Skip to content
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-6982] add native python exasol support #7621

Merged
merged 7 commits into from
Apr 2, 2020

Conversation

inytar
Copy link
Contributor

@inytar inytar commented Mar 4, 2020

This adds exasol DB support, including a hook, connection type &
operator. The pyexasol library is
used to interact with the database.


Issue link: AIRFLOW-6982

Make sure to mark the boxes below before creating PR: [x]

  • Description above provides context of the change
  • Commit message/PR title starts with [AIRFLOW-NNNN]. AIRFLOW-NNNN = JIRA ID*
  • Unit tests coverage for changes (not needed for documentation changes)
  • Commits follow "How to write a good git commit message"
  • Relevant documentation is updated including usage instructions.
  • I will engage committers as explained in Contribution Workflow Example.

* For document-only changes commit message can start with [AIRFLOW-XXXX].


In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.

setup.py Outdated Show resolved Hide resolved
@inytar
Copy link
Contributor Author

inytar commented Mar 6, 2020

I've looked into the lint & test issues. I've got Jenkins almost happy, except for one warning in the build-docs. I get the the following message:

opt/airflow/docs/_api/airflow/providers/exasol/hooks/exasol/index.rst:122: WARNING: Field list ends without a blank line; unexpected unindent.

I'm assuming it has to do with the doc-strings in airflow/providers/exasol/hooks/exasol.py, but I can't find what it is. I'd love some input.

@inytar inytar requested a review from mik-laj March 6, 2020 12:57
This adds exasol DB support, including a hook, connection type &
operator. The [pyexasol](https://github.com/badoo/pyexasol) library is
used to interact with the database.
@dimberman dimberman merged commit 69dc91b into apache:master Apr 2, 2020
@boring-cyborg
Copy link

boring-cyborg bot commented Apr 2, 2020

Awesome work, congrats on your first merged pull request!

@dimberman dimberman added this to the Airflow 2.0.0 milestone Apr 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants