Update libraries to modern ones#599
Open
dwoot-verily wants to merge 7 commits intotwitter:developfrom
Open
Conversation
- Update release version to 24.8.0-SNAPSHOT for util, finagle, scrooge, twitter-server - Add CLAUDE.md with codebase documentation for Claude Code - Fix missing info import in EmbeddedThriftServer - Simplify StatsReceiver scope calls to use single scope() with multiple args - All 1,390 tests passing with Java 21 and Scala 2.13.18
Revert to using hierarchicalScope and dimensionalScope APIs now that we have the latest finagle 24.8.0-SNAPSHOT build which includes proper dimensional metrics support. This provides both hierarchical (backward compatible) and dimensional (modern) metrics with labels for better observability. Changes: - ExceptionManager: Use hierarchicalScope/dimensionalScope/label for exception metrics - StatsFilter: Use hierarchicalScope/dimensionalScope/label for route metrics All 1,390 tests passing.
- Upgrade Guice from 5.1.0 to 7.0.0 (latest) - Upgrade scala-guice from 5.1.0 to 7.0.0 - Migrate from javax.inject to jakarta.inject namespace (201 occurrences across all files) - Replace javax.inject dependency with jakarta.inject-api 2.0.1 - Upgrade Jackson from 2.18.6 to 2.21.2 with annotations 2.21 - Upgrade Logback from previous version to 1.5.32 (security fixes) - Upgrade SLF4J to 2.0.17 (security fixes) - Upgrade Mustache to 0.9.14 - Fix ScalaObjectHandler for mustache.java 0.9.14 API changes (Array[AnyRef] -> java.util.List[Object]) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Update release version from 24.8.0-SNAPSHOT to 24.9.0-SNAPSHOT with comprehensive CHANGELOG entries covering all library upgrades: Thrift 0.22.0, Jackson 2.21.2, Guice 7.0.0, SLF4J 2.0.17, and other dependencies. Include upgrade documentation and test results. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
|
|
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
2 participants
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.