Skip to content

Update the sourceCatalogId field when the schema is updated #12505

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

Merged
merged 5 commits into from
May 4, 2022

Conversation

malikdiarra
Copy link
Contributor

When a user press the refresh schema button, and the save any changes they make to a connection configuration, we make sure we also update the identifier of the catalog that was used during the configuration update

@malikdiarra malikdiarra requested a review from a team as a code owner May 2, 2022 04:35
@malikdiarra malikdiarra requested a review from pmossman May 2, 2022 04:35
@github-actions github-actions bot added area/api Related to the api area/documentation Improvements or additions to documentation area/frontend area/platform issues related to the platform area/server area/worker Related to worker labels May 2, 2022
@malikdiarra malikdiarra temporarily deployed to more-secrets May 2, 2022 04:35 Inactive
@malikdiarra malikdiarra temporarily deployed to more-secrets May 2, 2022 04:35 Inactive
@malikdiarra malikdiarra temporarily deployed to more-secrets May 2, 2022 13:35 Inactive
@malikdiarra malikdiarra temporarily deployed to more-secrets May 2, 2022 13:35 Inactive
@alafanechere alafanechere temporarily deployed to more-secrets May 2, 2022 16:26 Inactive
@alafanechere alafanechere temporarily deployed to more-secrets May 2, 2022 16:26 Inactive
@malikdiarra malikdiarra merged commit dda50e1 into master May 4, 2022
@malikdiarra malikdiarra deleted the malik/update-source-catalog branch May 4, 2022 14:57
suhomud pushed a commit that referenced this pull request May 23, 2022
* Set SourceCatalogId during connectionUpdate operation

* Return catalogId when get a connection

* Fix db operation of standardSync.sourceCatalogId

- value is not set correctly during update operation
- value is not read

* UI modification to set the sourceCatalogId

* remove sourceCatalogId from diff computation

Co-authored-by: alafanechere <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api Related to the api area/documentation Improvements or additions to documentation area/platform issues related to the platform area/server area/worker Related to worker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants