-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AIRFLOW-5730] Enable get_pandas_df on PinotDbApiHook #6399
Conversation
Sorry, I submitted a wrong PR mistakenly. Will update it later. |
Same issue as #6057 ? |
Oh, Thanks! I missed that issue. I'll change my JIRA issue and this PR only about Pinot. |
Currently, PinotDbApiHook is disabling its get_pandas_df method by raising NotImplementedError. But it actually works as inherited from DbApiHook. This PR enables it.
Updated the PR. The CI failure seems to be a transient one. The newly added test works on my local environment, as follows:
|
Thanks @OmerJog for taking care of this PR, rebased on master to run the CI again :) |
Codecov Report
@@ Coverage Diff @@
## master #6399 +/- ##
==========================================
- Coverage 84.08% 83.77% -0.31%
==========================================
Files 635 635
Lines 36849 36847 -2
==========================================
- Hits 30983 30868 -115
- Misses 5866 5979 +113
Continue to review full report at Codecov.
|
(cherry picked from commit 8f1a585)
(cherry picked from commit 8f1a585)
(cherry picked from commit 8f1a585)
Make sure you have checked all steps below.
Jira
Description
Currently, DruidDbApiHook and PinotDbApiHook disable their get_pandas_df
methods by raising NotImplementedError. But they actually work as
inherited from DbApiHook. This PR enables them.
Tests
TestDruidHook.test_get_pandas_df in tests/hooks/test_druid_hook.py
TestPinotDbApiHook.test_get_pandas_df in tests/contrib/hooks/test_pinot_hook.py
Commits
Documentation