You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@seatunnel.apache.org by GitBox <gi...@apache.org> on 2022/03/31 16:42:06 UTC

[GitHub] [incubator-seatunnel] Rianico opened a new issue #1639: [Feature][connector] Upgrade the version of kudu from 1.7 to 1.12 and add units test for kudu

Rianico opened a new issue #1639:
URL: https://github.com/apache/incubator-seatunnel/issues/1639


   ### Search before asking
   
   - [X] I had searched in the [feature](https://github.com/apache/incubator-seatunnel/issues?q=is%3Aissue+label%3A%22Feature%22) and found no similar feature requirement.
   
   
   ### Description
   
   According to the [email](https://lists.apache.org/thread/gmlqqyw920o8xbvz38q5nton3kfhb278) , I want to upgrade the version of  kudu from 1.7 to 1.12 and add units test for kudu by using kudu-binary which can start a minimize kudu server but has a platform limitation(only support linxu-x86_64 and osx-x86_64).
   
   In order to solve the platform problem about the import problem of kudu-binary jar, maybe we can add profile which can be triggered according to the os environment automically. In this way, we don't have to add `-D` in maven or set `-D` in ide to solve the import problem of kudu-binary jar. But we still need a way to skip the unit test according to to OS version.
   
   Junit5 provides the annotation to skip the test according to the os, and more elegant semantics. But I'm not sure about whether it's worth upgrading the Junit from Junit4 to Junit5?  Anyone give some suggesstions?
   
   
   ### Usage Scenario
   
   Start the minimize kudu server in unit test, so we can test kudu-source, kudu-sink on a kudu cluster.
   
   ### Related issues
   
   #1459 
   
   ### Are you willing to submit a PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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-unsubscribe@seatunnel.apache.org

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