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

Sql doc update because of the failure with external oracle database #… #3272

Merged

Conversation

JiriOndrusek
Copy link
Contributor

Doc update for (does not fix the issue) #3256

@JiriOndrusek JiriOndrusek force-pushed the 3256-sql-oracle-native-limitations branch from 0e198cf to da6fc1c Compare November 5, 2021 13:43
Copy link
Contributor

@ppalaga ppalaga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some wording suggestions inline.

extensions/sql/runtime/src/main/doc/limitations.adoc Outdated Show resolved Hide resolved
extensions/sql/runtime/src/main/doc/limitations.adoc Outdated Show resolved Hide resolved
@aldettinger
Copy link
Contributor

@JiriOndrusek A small up on this PR. My understanding is that it just needs few fixes ? Or am I missing some context ?

@JiriOndrusek
Copy link
Contributor Author

JiriOndrusek commented Dec 13, 2021

@aldettinger Yes you are right, only small fixes (mainly doc) is need to be done. Unfortunately I'm not sure when I'll be able to fix it (maybe end of this week). But feel free to fix those small issues if you have a free capacity.

edited:

I should fix it today.

@JiriOndrusek JiriOndrusek force-pushed the 3256-sql-oracle-native-limitations branch 2 times, most recently from beb42f5 to 2af16c1 Compare December 14, 2021 08:20
@JiriOndrusek JiriOndrusek force-pushed the 3256-sql-oracle-native-limitations branch from 2af16c1 to 7e9035c Compare December 14, 2021 08:22
@JiriOndrusek
Copy link
Contributor Author

@aldettinger PR is fixed, it could be merged.

@ppalaga ppalaga merged commit cf55c4f into apache:main Dec 14, 2021
@aldettinger
Copy link
Contributor

@JiriOndrusek @ppalaga Thanks all.

Sorry for this late comment. It turns out that quarkusio/quarkus#20396 was fixed in the mean time. My understanding is that we still have #3256 to manage it on our side, so that's ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants