zhengxiang378928908-code commented on PR #12882: URL: https://github.com/apache/seatunnel/pull/12882#issuecomment-6308708877
@SEZ9 Thanks. All three are covered in #12882: - `NVARCHAR2` with `CHAR_LENGTH = 0`: every `NVARCHAR2` row in `DamengCatalogTest` has `CHAR_LENGTH = 0`, and a separate case covers a missing `LENGTH$` falling back to `DATA_LENGTH`. - Read-only user: `JdbcDmIT#testNationalCharLengthFromCatalog` creates `E2E_DM_READER` with only `GRANT SELECT` on the test table and reads the catalog as that user. - CHAR/VARCHAR/NCHAR: unchanged, asserted in both the catalog tests and the E2E. CI is still red only because of #12874 (fix in #12875); I'll sync dev and rerun the failed jobs once #12875 is merged. -- 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]
