FrankChen021 commented on PR #20286: URL: https://github.com/apache/druid/pull/20286#issuecomment-5585889481
Exact-head compatibility and CI receipt for base 33f7f778dfc2d8b06ca7c1d7a93f54f162526cee and head 92b56de668c77a012741dae3f7cc2a729d5b3092. Release inventory: the only actually published release transition in scope is KCL v3.5.1 (2026-07-22) to v3.5.2 (2026-09-04); there are no published intermediate releases. Sources: https://github.com/awslabs/amazon-kinesis-client/releases/tag/v3.5.1, https://github.com/awslabs/amazon-kinesis-client/releases/tag/v3.5.2, and https://github.com/awslabs/amazon-kinesis-client/compare/v3.5.1...v3.5.2. The carried-forward KCL 3.5.0/3.5.1 single-table migration, coordinator, lease, and migration-tool history remains Druid-irrelevant: this extension imports only retrieval.KinesisClientRecord and retrieval.AggregatorUtil for KPL deaggregation, not the scheduler, coordinator, lease, or migration components. Compatibility result for the exact head: | Area | Evidence | Result | | --- | --- | --- | | API/ABI | KinesisClientRecord and AggregatorUtil public signatures match in the 3.5.1 and 3.5.2 artifacts; neither retrieval class changed upstream. | SAFE | | Runtime | The 3.5.2 production changes are limited to coordinator, shard-sync, lease-assignment, migration retry, and migration-tool paths unused by Druid; KinesisRecordSupplier keeps the same AWS Record -> KinesisClientRecord -> AggregatorUtil flow. | SAFE | | Configuration | The PR changes only the Maven KCL version and license metadata; no Druid or KCL runtime setting is added or changed. | SAFE | | Serialization/wire | AggregatorUtil, KinesisClientRecord, KPL deaggregation, Kinesis wire calls, and Druid JSON formats are unchanged. | SAFE | | Persistence | Druid does not persist KCL coordinator, lease, migration, or KinesisClientRecord state; the single-table migration is not exercised by this extension. | SAFE | | Clients | Druid continues to use its managed AWS SDK v2 Kinesis client; the KCL bump does not alter the Kinesis service contract used by Druid. | SAFE | | Transitive dependencies | Both published KCL POMs and parent POMs differ only by the version value; exact Maven resolution selects KCL 3.5.2 with Druid-managed AWS SDK 2.54.10, Guava 33.7.1-jre, protobuf 4.36.1, and commons-lang3 3.20.0, with existing exclusions intact. | SAFE | | License | The KCL 3.5.2 POM remains Apache License 2.0 and licenses.yaml records version 3.5.2. | SAFE | | Extension/plugin SPI | No Druid Java, service-loader, extension, or plugin SPI file changes; only dependency metadata changes. | SAFE | PR-local checks: the complete two-file diff was reviewed; git diff --check passed; focused Maven dependency-tree resolution passed; the existing license fix is present in head 92b56de668. No additional PR-local fix, commit, push, or failed-job rerun was needed. GitHub state at the exact head: base is still 33f7f778dfc2d8b06ca7c1d7a93f54f162526cee, 24 check runs are completed SUCCESS, and unit tests / coverage-jacoco / coverage-jacoco remains QUEUED. No failed job is eligible for rerun, and the queued job is not rerunnable as a completed failure. Final state: compatibility is SAFE, but the PR is left OPEN and unapproved because the exact head is not fully terminal-green. Next action is to let coverage-jacoco complete, then recheck head 92b56de668c77a012741dae3f7cc2a729d5b3092 and approve only if it remains unchanged and successful. No merge performed. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
