liaorui opened a new pull request, #6741:
URL: https://github.com/apache/inlong/pull/6741

   ### Prepare a Pull Request
   *(Change the title refer to the following example)*
   
   - Title Example: [INLONG-XYZ][Component] Title of the pull request
   
   *(The following *XYZ* should be replaced by the actual [GitHub 
Issue](https://github.com/apache/inlong/issues) number)*
   
   - Fixes #6740
   
   ### Motivation
   
   *Explain here the context, and why you're making that change. What is the 
problem you're trying to solve?*
   
   Inspired by doris-cdc-inlong, starrocks-cdc-inlong can also support 
transferring all tables in one database on time.
   
   StarRocks FE api has changed, been different from Doris FE api. If you use 
doris-cdc-inlong to handle StarRocks tables writing, it will give a error: 
http://x.x.x.x:xxxx/api/backends?is_alive=true failed. java.io.IOException: 
Failed to get response from Doris. That is why we need a sarrocks-cdc-inlong 
connector.
   
   Based on starrocks-cdc of community,  starrocks-cdc-inlong has improved all 
tables writing in one database one time.
   
   ### Modifications
   
   *Describe the modifications you've done.*
   
   1.  StarRocksDynamicSinkFunction's invoke method supports handling debezium 
canal-json upstream parsing. It picks data from upstream, assembles data with 
json format.
   
   2. StarRocksSinkManager's init method will only validate table structure in 
single table transferring, and skip validating in all database migration.
   
   3. StarRocksStreamLoadVisitor's checkLabelState method add `String database` 
parameter.
   
   4. StarRocksSinkManager removes checkFlushException method invoking in 
waitAsyncFlushingDone, close and flushInternal methods. If there is a error 
when writing to StarRocks, flink cdc will fail to make a checkpoint because of 
this error.
   
   ### Verifying this change
   
   *(Please pick either of the following options)*
   
   - [ ] This change is a trivial rework/code cleanup without any test coverage.
   
   - [ ] This change is already covered by existing tests, such as:
     *(please describe tests)*
   
   - [ ] This change added tests and can be verified as follows:
   
     *(example:)*
     - *Added integration tests for end-to-end deployment with large payloads 
(10MB)*
     - *Extended integration test for recovery after broker failure*
   
   ### Documentation
   
     - Does this pull request introduce a new feature? (yes / no)
     - If yes, how is the feature documented? (not applicable / docs / JavaDocs 
/ not documented)
     - If a feature is not applicable for documentation, explain why?
     - If a feature is not documented yet in this PR, please create a follow-up 
issue for adding the documentation
   


-- 
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: commits-unsubscr...@inlong.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to