Skip to content

[DMS-556] Adding Cast Hierarchy To Int64 for DMS#16

Closed
JBrenesSimpat wants to merge 1 commit intomainfrom
DMS-556
Closed

[DMS-556] Adding Cast Hierarchy To Int64 for DMS#16
JBrenesSimpat wants to merge 1 commit intomainfrom
DMS-556

Conversation

@JBrenesSimpat
Copy link
Contributor

No description provided.

@CSR2017
Copy link
Contributor

CSR2017 commented Mar 14, 2025

@JBrenesSimpat
Can we try adding new transforms to the connector file: https://github.com/Ed-Fi-Alliance-OSS/Data-Management-Service/blob/main/eng/docker-compose/postgresql_connector.json. I noticed that there is already an existing transform for type casting. Here’s the documentation for reference: Confluent Cast Transform. I guess we can apply this to specific topic/ table using the predicate.

@stephenfuqua
Copy link
Contributor

... Here’s the documentation for reference: Confluent Cast Transform. I guess we can apply this to specific topic/ table using the predicate.

I was a little worried when I saw this comment: the Confluence source code is not open sourced and we try to avoid it. However, the transform is actually part of the core Kafka source code under the Apache license: Cast.java. All is well then! We're in good shape. This is just a good reminder that we want to avoid any transform that has confluent in the class namespace.

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.

3 participants