Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23609 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/23609 Tested-by: Kudu Jenkins Reviewed-by: Marton Greber <[email protected]> Reviewed-by: Zoltan Chovan <[email protected]> --- 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(-) Approvals: Kudu Jenkins: Verified Marton Greber: Looks good to me, approved Zoltan Chovan: Looks good to me, but someone else must approve -- 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: merged Gerrit-Change-Id: I3fcedb050ce4deea1c317b3cf3687502cbe81ce7 Gerrit-Change-Number: 23609 Gerrit-PatchSet: 6 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]> Gerrit-Reviewer: Zoltan Chovan <[email protected]>
