VGalaxies commented on PR #17926: URL: https://github.com/apache/iotdb/pull/17926#issuecomment-4689955918
CI follow-up:\n\n- Investigated the previous `dual-tree-auto-enhanced (17, HighPerformanceMode, HighPerformanceMode, ubuntu-latest, 0)` failure. It failed in `IoTDBPipeClusterIT.testRegisteringNewDataNodeAfterTransferringData` with inconsistent row counts across DataNodes, which is unrelated to the subscription TsFile termination-drain change.\n- Pushed `ddd09219b2` to address the Sonar/checkstyle findings on this PR: removed the non-primitive volatile field access pattern for `currentTerminateEvent`, fixed the long log line, and placed overloaded methods next to each other.\n- Local verification passed:\n - `mvn spotless:apply -pl iotdb-core/datanode`\n - `mvn checkstyle:check -pl iotdb-core/datanode`\n - `mvn verify -DskipUTs -Dit.test=org.apache.iotdb.subscription.it.dual.tablemodel.IoTDBSubscriptionTopicIT#testTsFileSnapshotDrainsPendingBatchBeforeTermination -DfailIfNoTests=false -Dfailsafe.failIfNoSpecifiedTests=false -pl integration-test -am -P MultiClusterIT2Subscriptio nTableArchVerification -P with-integration-tests`\n- The new CI run on `ddd09219b2` has already passed `SonarCloud Code Analysis` and `subscription-table-arch-verification`; the remaining large matrix jobs and Codecov are still pending. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
