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.

Sign up for GitHub

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

Jump to bottom

[Task]: Migrate remaining io.jdbc.JdbcLogicalType to Portable Logical Types in core.schemas.logicaltypes #23526

Closed
Abacn opened this issue Oct 6, 2022 · 1 comment · Fixed by #23548
Closed

[Task]: Migrate remaining io.jdbc.JdbcLogicalType to Portable Logical Types in core.schemas.logicaltypes #23526

Abacn opened this issue Oct 6, 2022 · 1 comment · Fixed by #23548
Assignees
@Abacn
Labels
io java jdbc P2 task
Milestone
2.44.0 Release

Comments

@Abacn
Copy link
Contributor

Abacn commented Oct 6, 2022

What needs to happen?

After #23014 there are following logical types in io.jdbc.LogicalTypes:

  • FixedLengthString
  • FixedLengthBytes
  • VariableLengthString
  • VariableLengthBytes

We should migrate them to portable logical type in order to use them in cross-language scenario. For example, currently Python cross-language JDBC Read cannot read rows containing fixed or variable length string: https://stackoverflow.com/questions/68758361/error-beamlogical-typejavasdkv1-while-using-apache-beam-io-jdbc-readfromjdbc

Issue Priority

Priority: 2

Issue Component

Component: io-java-jdbc
@Abacn
Copy link
Contributor Author

Abacn commented Oct 8, 2022

.take-issue

@Abacn Abacn mentioned this issue Oct 8, 2022
Merged
4 tasks
@github-actions github-actions bot added this to the 2.44.0 Release milestone Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Abacn Abacn

Labels
io java jdbc P2 task
Projects
None yet
Milestone
2.44.0 Release
Development

Successfully merging a pull request may close this issue.

Migrate BINARY, VARBINARY, CHAR, VARCHAR jdbc logical types to portable Abacn/beam
1 participant
@Abacn