-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Fix parsing of DBRef MongoDB objects #10276
Conversation
| 0.1.1 | 2021-09-21 | [6364](https://github.com/airbytehq/airbyte/pull/6364) | Source MongoDb: added support via TLS/SSL | | ||
| 0.1.0 | 2021-08-30 | [5530](https://github.com/airbytehq/airbyte/pull/5530) | New source: MongoDb ported to java | | ||
|:--------| :--- | :--- | :--- | | ||
| 0.1.11 | 2022-02-11 | [10276](https://github.com/airbytehq/airbyte/pull/10276) | Create a custom codec registry to handle DBRef MongoDB objects | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 0.1.11 | 2022-02-11 | [10276](https://github.com/airbytehq/airbyte/pull/10276) | Create a custom codec registry to handle DBRef MongoDB objects | | |
| 0.1.12 | 2022-02-11 | [10276](https://github.com/airbytehq/airbyte/pull/10276) | Create a custom codec registry to handle DBRef MongoDB objects | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I confirm that acceptance test are passing. I made minor changes to bump to the right version.
Thanks @NahidOulmi for this contrib and glad it fixes your original problem. |
What
Related to this issue : #9604
The default codec registry does not handle MongoDB DBRef objects.
This PR provides a fix to this issue.
How
The Mongo team provides a DBRefCodecProvider for handling DBRef objects.
This PR provides a custom codec registry that is derived from the default codec registry.
Only difference is the added DBRefCodecProvider in here.
Community member or Airbyter
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.Integration & acceptation tests passed:
![image](https://user-images.githubusercontent.com/92733427/153624420-402fb3dd-5f9c-4d6b-a819-006142a4019f.png)