[AIRFLOW-2591] Set default value of autocommit to False in DbApiHook.run()

[imroc]

JIRA

• My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"
  • https://issues.apache.org/jira/browse/AIRFLOW-2591

Description

• Here are some details about my PR, including screenshots of any UI changes:
  In previous PR AIRFLOW-2581, I changed the default value of
  autocommit to True in DbApiHook.run(), which may cause some performance
  issues, so I changed it back to False.

Commits

• My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":
  1. Subject is separated from body by a blank line
  2. Subject is limited to 50 characters
  3. Subject does not end with a period
  4. Subject uses the imperative mood ("add", not "adding")
  5. Body wraps at 72 characters
  6. Body explains "what" and "why", not "how"

[imroc]

@Fokko @yrqls21 PTAL. In my previous PR AIRFLOW-2581, I changed the default value of
autocommit to True in DbApiHook.run(), which may cause some performance
issues, so I changed it back to False.

[KevinYang21]

LGTM, ty for the quick turnaround.