zhangshenghang opened a new pull request, #11220:
URL: https://github.com/apache/seatunnel/pull/11220

   ### Purpose
   
   Fixes #9966.
   
   Kafka topics can legally contain multiple dots, for example 
`reg.country.user_activity.activity`. SeaTunnel can create a `TablePath` with 
that full topic as an opaque table name, but later code may parse the string 
form again through `TablePath.of(String)`. The previous parser rejected names 
with more than three dot-separated parts, which caused jobs using these topic 
names to fail with:
   
   `Cannot get split 'reg.country.user_activity.activity' to get databaseName 
and tableName`
   
   ### Changes
   
   - Keep the existing parsing behavior for one, two, and three-part table 
paths.
   - Treat names with more than three dot-separated parts as opaque table names 
instead of rejecting them.
   - Add API coverage for dotted opaque table names.
   - Add Kafka source config coverage for a topic with multiple dots.
   
   ### Tests
   
   - `./mvnw spotless:apply`
   - `git diff --check`
   - `./mvnw -pl seatunnel-api -Dtest=TablePathTest test -Dskip.spotless=true`
   - `./mvnw -pl seatunnel-api install -DskipTests -Dskip.spotless=true`
   - `./mvnw -pl seatunnel-connectors-v2/connector-kafka 
-Dtest=KafkaSourceConfigTest test -Dskip.spotless=true`
   
   Note: `./mvnw -q -DskipTests verify` was started but produced no output for 
an extended period and was interrupted locally. A narrower `-am` verification 
attempt was blocked by an unrelated local `seatunnel-config-shade` compilation 
issue.
   


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