xiangfu0 opened a new pull request, #18001:
URL: https://github.com/apache/pinot/pull/18001

   ## Summary
   - Remove 12 controller config constants deprecated since v0.8.0 (2020-2021)
   - Remove their fallback getter methods that resolved old keys to new keys
   - Update PinotConfigUtils to use new config format with period strings
   - All replacements have been in production use since v0.8.0
   
   Constants removed include 
`DEPRECATED_RETENTION_MANAGER_FREQUENCY_IN_SECONDS`, 
`DEPRECATED_OFFLINE_SEGMENT_INTERVAL_CHECKER_FREQUENCY_IN_SECONDS`, and 10 
others. Each had an explicit replacement that is already the primary config 
path. Pinot is now in the 1.x line — no cluster should still be sending 
0.8.0-era config keys.
   
   ## Test plan
   - [x] Verified no non-test code references the deprecated constants
   - [x] `./mvnw spotless:apply -pl pinot-controller` passes
   - [x] `./mvnw checkstyle:check -pl pinot-controller` passes (0 violations)
   - [x] `./mvnw license:check -pl pinot-controller` passes
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

Reply via email to