[AIRFLOW-2412] Fix HiveCliHook.load_file to address HIVE-10541

[sekikn]

Make sure you have checked all steps below.

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-2412
  • In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

• Here are some details about my PR, including screenshots of any UI changes:

HiveCliHook.load_file doesn't actually execute
LOAD DATA statement via beeline bundled with
Hive under 2.0 due to HIVE-10541.
This PR provides a workaround for this problem.

Tests

• My PR adds the following unit tests OR does not need testing for this extremely good reason:

This PR adds the following tests. In addition, I confirmed the problem described above is resolved using a real Hadoop cluster with Hive 1.2.1.

• TestHiveCliHook.test_run_cli
• TestHiveCliHook.test_load_file

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"

Documentation

• In case of new functionality, my PR adds documentation that describes how to use it.
  • When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

• Passes git diff upstream/master -u -- "*.py" | flake8 --diff

[codecov-io]

Codecov Report

Merging #3327 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3327      +/-   ##
==========================================
- Coverage   75.89%   75.88%   -0.01%     
==========================================
  Files         197      197              
  Lines       14729    14730       +1     
==========================================
  Hits        11178    11178              
- Misses       3551     3552       +1

Impacted Files	Coverage Δ
airflow/hooks/hive_hooks.py	57.92% <100%> (+0.12%)	⬆️
airflow/models.py	86.64% <0%> (-0.05%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9eb52c...7ec74bd. Read the comment docs.

[Fokko]

@gglanzani Can you take a look? I'm not a hive expert

[gglanzani]

@Fokko This seem as a pretty innocent PR to me, adding only a newline at the end of the SQL.

The beeline bug is indeed real and, although it has been addressed in newer versions of beeline, it still affects most clusters.

Since this doesn't functionally change anything, I'd +1 it.

[Fokko]

Thanks @sekikn and @gglanzani

LGTM, merging to master and v1-10

[sekikn]

@Fokko @gglanzani Thanks!