Hello Marton Greber, Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23609
to look at the new patch set (#5).
Change subject: [java] Split TestKuduClient into smaller modules
......................................................................
[java] Split TestKuduClient into smaller modules
The monolithic TestKuduClient class (~2600 lines with 50 tests) was
causing regular timeouts on TSAN builds. This commit splits it into
6 focused test modules that can run independently, significantly
reducing timeout risk on slow builds.
Test modules:
- TestKuduClientBasic (7 tests): Basic table operations, timestamps,
and soft delete functionality
- TestKuduClientDataTypes (8 tests): Data type tests including strings,
VARCHAR, binary, timestamps, dates, and decimals
- TestKuduClientScanner (7 tests): Scanner lifecycle, expiration,
keep-alive, limits, predicates, and non-covered ranges
- TestKuduClientAdvanced (8 tests): Auto-incrementing columns, array
types, and custom hash schemas per range partition
- TestKuduClientConcurrency (10 tests): Concurrent operations,
read-your-writes consistency, concurrent flush, and session lifecycle
- TestKuduClientSecurity (10 tests): Authentication, certificates,
location assignment, and cluster ID
The original TestKuduClient.java has been removed entirely. CI/Jenkins
should run the individual test classes rather than a suite wrapper.
Verified with dist-test: 1000 runs of each individual module complete
successfully without timeouts.
Change-Id: I3fcedb050ce4deea1c317b3cf3687502cbe81ce7
---
M java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java
M
java/kudu-client/src/test/java/org/apache/kudu/client/TestFlexiblePartitioning.java
D java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
A
java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClientAdvanced.java
A java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClientBasic.java
A
java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClientConcurrency.java
A
java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClientDataTypes.java
A
java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClientScanner.java
A
java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClientSecurity.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestScanToken.java
10 files changed, 3,040 insertions(+), 2,624 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/09/23609/5
--
To view, visit http://gerrit.cloudera.org:8080/23609
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3fcedb050ce4deea1c317b3cf3687502cbe81ce7
Gerrit-Change-Number: 23609
Gerrit-PatchSet: 5
Gerrit-Owner: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
