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

Unregister MongoDb's ChangeStreamDocument from reflection once it is provided by Quarkus #2649

Closed
JiriOndrusek opened this issue May 24, 2021 · 2 comments · Fixed by #2759
Closed
Assignees
Milestone

Comments

@JiriOndrusek
Copy link
Contributor

Follows #2622
See more details in #2642 (comment).

Quarkus mongodb extension does not contain reflection registration of ChangeStreamDocument. It is reported as quarkusio/quarkus#17432

Once quarkusio/quarkus#17432 is fixed, registration in MongoDbProcessor could be removed.

@JiriOndrusek JiriOndrusek self-assigned this May 24, 2021
@JiriOndrusek JiriOndrusek changed the title MongoDb test contains workaround, which should be removed once quarkus mongodb extension is fixed MongoDb extension contains workaround, which should be removed once quarkus mongodb extension is fixed May 24, 2021
@ppalaga
Copy link
Contributor

ppalaga commented May 26, 2021

The blocking issue seems to be fixed. Let's wait for the next Quarkus milestone.

@ppalaga ppalaga added this to the 2.0.0 milestone May 26, 2021
@ppalaga ppalaga changed the title MongoDb extension contains workaround, which should be removed once quarkus mongodb extension is fixed Unregister MongoDb's ChangeStreamDocument from reflection once it is provided by Quarkus May 26, 2021
@ppalaga ppalaga modified the milestones: 2.0.0, 2.0.0-M2 Jun 4, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 9, 2021
@ppalaga
Copy link
Contributor

ppalaga commented Jun 9, 2021

A fix is underway.

ppalaga added a commit that referenced this issue Jun 10, 2021
zhfeng pushed a commit to zhfeng/camel-quarkus that referenced this issue Jun 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants