Open
Conversation
- Update maven.compiler.source/target from 1.6 to 17 - Update maven-compiler-plugin to 3.11.0 with Java 17 target - Update maven-surefire-plugin to 3.2.5 with --add-opens JVM args for Java 17 module system compatibility - Remove deprecated Twitter/Typesafe Maven repositories Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add hadoop.version property set to 3.3.6
- Update all Hadoop dependencies to use ${hadoop.version}:
- hadoop-common
- hadoop-hdfs-client
- hadoop-mapreduce-client-core
- hadoop-aws
- hadoop-openstack
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Update parquet.version property from 1.9.0 to 1.17.0
- Update parquet-avro dependency to use ${parquet.version}
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add protobuf.version property set to 4.29.3 - Update protobuf-java and protobuf-java-util dependencies - Replace protoc-jar-maven-plugin with protobuf-maven-plugin (org.xolstice) which supports Protobuf 4.x - Add os-maven-plugin extension for OS-specific protoc binary detection - Update ProtobufUtil.java: GeneratedMessageV3 → GeneratedMessage (class renamed in Protobuf 4.x) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Update orc-core from 1.6.2 to 2.2.2 - Add hive-storage-api 2.8.1 dependency required by ORC 2.x ORC 2.x is required for Protobuf 4.x compatibility. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Replace mockito-all 1.9.5 with mockito-core 5.11.0 - Remove PowerMock dependencies (incompatible with Java 17) - Add mockito-inline 5.2.0 for static mocking support - Update JUnit from 4.11 to 4.13.2 Test file changes: - Convert PowerMock @RunWith/@PrepareForTest to Mockito mockStatic() - Replace org.mockito.Matchers with org.mockito.ArgumentMatchers - Update OstrichMetricCollectorTest with Mockito static mocking - Update FileRegistryTest with Mockito static mocking - Update UploaderTest with Mockito static mocking - Rewrite FileReaderWriterFactoryTest to use real file I/O - Update Avro/Parquet test files to remove PowerMock annotations - Update parser tests to remove PowerMock annotations - Update RebalanceHandlerTest for Mockito 5.x compatibility Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove classes that depend on old Kafka APIs removed in modern versions: - LegacyKafkaClient.java - LegacyKafkaMessageIterator.java - KafkaMessageTimestamp.java (interface) - KafkaMessageTimestampFactory.java - Kafka8MessageTimestamp.java - Kafka10MessageTimestamp.java - KafkaMessageTimestampFactoryTest.java These classes used kafka.javaapi.* and kafka.message.* packages which are no longer available in modern Kafka client libraries. Users should use SecorKafkaClient and SecorKafkaMessageIterator instead. Note: Configuration files still reference these classes but will need to be updated to use the modern implementations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Move mLocks.put() after successful acquire() and mLocks.remove() before release() to prevent stale entries that cause AssertionError during Kafka rebalance. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ColonelLanda
force-pushed
the
feature/upgrade_java_17
branch
from
675d534 to
2bb913e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.