zhang-arvin opened a new pull request, #11874: URL: https://github.com/apache/seatunnel/pull/11874
## Purpose Fixes #11685. `YashanDbTypeConverter` has cases for `CHAR`/`VARCHAR` and `NCHAR`/`NVARCHAR`, but not for the Oracle-compatible synonyms `VARCHAR2` and `NVARCHAR2`. Reading a YashanDB table containing such columns falls through to the `default` branch and fails: ``` ErrorCode:[COMMON-17], ErrorDescription:['YashanDB' unsupported convert type 'VARCHAR2' of 'test' to SeaTunnel data type.] ``` ## Changes - Added `VARCHAR2` and `NVARCHAR2` constants in `YashanDbTypeConverter` - Added `case VARCHAR2:` to the `CHAR`/`VARCHAR` branch (byte-length semantics) - Added `case NVARCHAR2:` to the `NCHAR`/`NVARCHAR` branch (char-length semantics) - Added unit tests for `VARCHAR2` and `NVARCHAR2` in `YashanDbTypeConverterTest` ## Testing ``` ./mvnw test -pl seatunnel-connectors-v2/connector-jdbc -Dtest=YashanDbTypeConverterTest Tests run: 32, Failures: 0, Errors: 0, Skipped: 0 ``` -- 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]
