Copilot commented on code in PR #17995:
URL: https://github.com/apache/pinot/pull/17995#discussion_r3003597162
##########
pom.xml:
##########
@@ -199,7 +199,7 @@
<spark3.version>3.5.8</spark3.version>
<kafka3.version>3.9.2</kafka3.version>
<kafka4.version>4.1.1</kafka4.version>
- <confluent.version>7.9.5</confluent.version>
+ <confluent.version>8.2.0</confluent.version>
Review Comment:
The PR updates `confluent.version` (7.9.5 -> 8.2.0) in addition to the
protobuf upgrade, but this dependency bump is not mentioned in the PR
title/summary. Since this property affects all `io.confluent:*` artifacts via
dependencyManagement, either document the rationale/impact in the PR
description or revert this change if it is not required for the protobuf 4.x
migration.
--
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]
