Skip to content

Conversation

@vchiapaikeo
Copy link
Contributor

@vchiapaikeo vchiapaikeo commented May 1, 2023

Closes: #24953

Add example documentation to Oracle operators. More specifically, clarify the correct usage of the OracleStoredProcedureOperator.

Just wondering, is there a way to build AND RUN the documentation site locally?


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@vchiapaikeo vchiapaikeo force-pushed the vchiapaikeo/oracle-fix-hook-v1 branch 7 times, most recently from 9002ab8 to 78c42d2 Compare May 6, 2023 00:22
@eladkal eladkal changed the title [oracle] Add docs for operators Add docs for Oracle operators May 6, 2023
@vchiapaikeo vchiapaikeo force-pushed the vchiapaikeo/oracle-fix-hook-v1 branch 2 times, most recently from ed1ad8a to 205fb05 Compare May 9, 2023 12:43
@vchiapaikeo vchiapaikeo force-pushed the vchiapaikeo/oracle-fix-hook-v1 branch from 205fb05 to 67900d0 Compare May 9, 2023 12:44
@potiuk potiuk merged commit edebfe3 into apache:main May 9, 2023
potiuk added a commit to potiuk/airflow that referenced this pull request May 9, 2023
The apache#30979 added examples for Oracle, but the examples used
deprecated imports to OracleOperator (and it has not been caught as
provider tests did not run on changing just examples).

This PR changes the import to non-deprecated SQLExecuteQueryOperator
potiuk added a commit that referenced this pull request May 9, 2023
The #30979 added examples for Oracle, but the examples used
deprecated imports to OracleOperator (and it has not been caught as
provider tests did not run on changing just examples).

This PR changes the import to non-deprecated SQLExecuteQueryOperator
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.

oracle hook _map_param() incorrect

3 participants