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

RD-10439 data duplication on MySQL.InferAndRead #338

Merged
merged 7 commits into from
Jan 29, 2024
Merged

RD-10439 data duplication on MySQL.InferAndRead #338

merged 7 commits into from
Jan 29, 2024

Conversation

torcato
Copy link
Contributor

@torcato torcato commented Jan 24, 2024

The problem was not using the database when asking the metadata for tables.

I had also to change the RDBMS clients as we were using a lot of times the credential name instead of the database base name.
So added a val database: Option[String] to the base jdbc client and changed the way of inferring.

@torcato
Copy link
Contributor Author

torcato commented Jan 24, 2024

/publish

@torcato
Copy link
Contributor Author

torcato commented Jan 24, 2024

/publish

will publish utils version: 0.28.15+2-f0bb732b-SNAPSHOT
will publish client version: 0.28.15+2-f0bb732b-SNAPSHOT
will publish snapi-parser version: 0.28.15+2-f0bb732b-SNAPSHOT
will publish snapi-frontend version: 0.28.15+2-f0bb732b-SNAPSHOT
will publish snapi-truffle version: 0.28.15+2-f0bb732b-SNAPSHOT
will publish snapi-client version: 0.28.15+2-f0bb732b-SNAPSHOT
will publish sql-client version: 0.28.15+2-f0bb732b-SNAPSHOT
published
run 7641726844 success ✔️

@torcato torcato marked this pull request as draft January 25, 2024 14:56
@torcato torcato changed the title data duplication on MySQL.InferAndRead RD-10439 data duplication on MySQL.InferAndRead Jan 25, 2024
The table and schema builders were using credential name instead of
datase in some cases
@torcato
Copy link
Contributor Author

torcato commented Jan 29, 2024

/publish

will publish utils version: 0.28.15+4-f60a3730-SNAPSHOT
will publish client version: 0.28.15+4-f60a3730-SNAPSHOT
will publish snapi-parser version: 0.28.15+4-f60a3730-SNAPSHOT
will publish snapi-frontend version: 0.28.15+4-f60a3730-SNAPSHOT
will publish snapi-truffle version: 0.28.15+4-f60a3730-SNAPSHOT
will publish snapi-client version: 0.28.15+4-f60a3730-SNAPSHOT
will publish sql-client version: 0.28.15+4-f60a3730-SNAPSHOT
published
run 7694709367 success ✔️

@torcato
Copy link
Contributor Author

torcato commented Jan 29, 2024

/publish

will publish utils version: 0.28.15+6-f8868e77-SNAPSHOT
will publish client version: 0.28.15+6-f8868e77-SNAPSHOT
will publish snapi-parser version: 0.28.15+6-f8868e77-SNAPSHOT
will publish snapi-frontend version: 0.28.15+6-f8868e77-SNAPSHOT
will publish snapi-truffle version: 0.28.15+6-f8868e77-SNAPSHOT
will publish snapi-client version: 0.28.15+6-f8868e77-SNAPSHOT
will publish sql-client version: 0.28.15+6-f8868e77-SNAPSHOT
published
run 7694916459 success ✔️

@torcato torcato marked this pull request as ready for review January 29, 2024 12:01
@torcato torcato requested a review from bgaidioz January 29, 2024 12:01
@torcato torcato merged commit f7a3df5 into main Jan 29, 2024
12 checks passed
@torcato torcato deleted the RD-10439 branch January 29, 2024 13:30
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.

2 participants