nielifeng commented on issue #11620:
URL: https://github.com/apache/seatunnel/issues/11620#issuecomment-5163420833

   Reviewed community PR #11631 for this issue. The implementation wires TEXT 
into validation and TextDeserializationSchema, and the existing dependency/API 
path is compatible.\n\nBefore it is mergeable, the PR still needs:\n- Spotless 
fixes in PulsarSource.java; the module spotless:check reproduces these 
failures.\n- A regression test for single-table TEXT plus custom 
field_delimiter, and preferably a tables_configs case.\n- The invalid-format 
error text updated to include TEXT.\n\nGitHub Build is currently failing 
because the contributor's fork has not enabled CI. I requested changes on 
#11631 and will recheck the PR after a new head/green checks.


-- 
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]

Reply via email to