Hello everyone,

I want to upgrade the version of Kudu from 1.7 to 1.12 and add some unit
tests for Kudu according to the previous discussion on github.

More infomation about the discussion can refer to this PR(
https://github.com/apache/incubator-seatunnel/pull/1459).

And now I want to confirm how about this idea?

Due to Kudu has only supported linux/macOS x86, there may be a little
problem when we run on Windows or Mac M1 aarch64:

- Maven: We need to set a fixed classifier which is different from the
Kudu's offcial example. We will get an error when import on Windows/Mac M1
aarch64 platform due to the ${os.detected.classifier} will specify the
infomation according to our platform, and we can't find the specified
version of jar. All versions of the jar that Kudu has supported can refer
to https://repo1.maven.org/maven2/org/apache/kudu/kudu-binary/1.12.0/.

```xml
  <!-- Windows and Mac M1 aarch64 will get error when import -->
  <dependency>
    <groupId>org.apache.kudu</groupId>
    <artifactId>kudu-binary</artifactId>
    <version>1.12.0</version>
    <classifier>${os.detected.classifier}</classifier>
    <scope>test</scope>
  </dependency>


  <!-- set a fixed classifier which is linux-x86_64 -->
  <dependency>
    <groupId>org.apache.kudu</groupId>
    <artifactId>kudu-binary</artifactId>
    <version>1.12.0</version>
    <classifier>linux-x86_64</classifier>
    <scope>test</scope>
  </dependency>
```

- Junit: We may need to upgrade the Junit version from Junit4 to Junit5 in
order to use the new annotation (@EnabledOnOs, which can make the unit test
only run on the specified platform), or we can also use the
assume(condition) in scalatest to achieve this goal.

So how about the idea which are mentioned above?
(PS: i am not so good at English. And this is the first time to use this
form of email, please forgive me if the description is not clear).

Best wishes!

Reply via email to