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

   Fixes [#11166 [Bug][Sort] Mongodb2StarRocksTest Failure Due to Potential 
Dependency Conflicts](https://github.com/apache/inlong/issues/11166)
   
   ### Motivation
   When running the `Mongodb2StarRocksTest` in the `sort-end-to-end-test-v1.15` 
module, the test failed due to a `NoClassDefFoundError` exception caused by a 
missing `org.mongodb.bson` module. 
   
   This issue arose because Pull Request #11136  added the `org.mongodb.bson` 
dependency in the `pom.xml` file of the project root directory, leading to a 
conflict with the existing `org.mongodb.bson` dependency in 
`inlong-sort/sort-flink/sort-flink-v1.15/sort-connectors/mongodb-cdc`. 
   
   Therefore, the problematic POM must be revised to limit the impact of the 
newly introduced BSON dependency.
   
   ### Modifications
   
   - Removed the `org.mongodb.bson` dependency from the `pom.xml` in the root 
directory.
   - Introduced the `org.mongodb.bson` dependency in the `pom.xml` file of 
`inlong-sdk/transform-sdk` to decrease the affecting range of the newly 
introduced dependency from Pull Request #11136.
   
   ### Verifying this change
   
   *(Please pick either of the following options)*
   
   - [ ] This change is a trivial rework/code cleanup without any test coverage.
   
   - [x] This change is already covered by existing tests, such as:
     `Mongodb2StarRocksTest `
   
   - [ ] This change added tests and can be verified as follows:
   
   
   ### Documentation
   
     - Does this pull request introduce a new feature? **No**
     - If yes, how is the feature documented? (not applicable / docs / JavaDocs 
/ not documented) **Not Applicable**
     - If a feature is not applicable for documentation, explain why? **Bug 
Fix. No new feature introduced**
   
   ### Additional Notes
   - It is advisable to run all existing tests in the module before creating PR 
to ensure compatibility across all dependencies and to prevent similar issues 
in the future, as GitHub action only runs Unit Tests for the edited part, not 
all UTs.
   


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