SEZ9 commented on issue #12586: URL: https://github.com/apache/seatunnel/issues/12586#issuecomment-5946661233
Thanks for the thorough report and the before/after matrix. The root cause analysis is clear: #11232 only maps unknown types to STRING when the sqlType is `Types.OTHER`, but `PostgresCatalog` builds the type define without a `sqlType`, and pgjdbc 42.4.3 / Debezium report enums as `Types.VARCHAR`, so the enum type names fall through to the unsupported-type error. Falling back to STRING for unrecognized type names in `PostgresTypeConverter` (not only for `Types.OTHER`) sounds like the right direction, and the CDC schema-change path will need the same handling so it doesn't hit COMMON-17. Since you have results "with fix" and are willing to submit a PR, please open one against `dev`. It would be great to include e2e coverage for an enum column in a non-public schema, a NOT NULL enum with a default, and NULL values, for both the Postgres-CDC and JDBC source paths. <!-- streview-comment:1470 --> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
