Vamsi-klu commented on PR #18979: URL: https://github.com/apache/pinot/pull/18979#issuecomment-5249500282
Pushed `5ea20f0`. All eight red checks had a single root cause: three `/** */` blocks in `JsonFunctions.java` where master now requires the `///` JEP 467 form. That failed checkstyle, which failed the pinot-common build, which took Linter, Unit Test Set 1, Quickstart, Binary Compatibility and all four compat-verifier jobs down with it. `./mvnw -pl pinot-common checkstyle:check` reports zero violations now. The three bot threads are addressed in the same push and answered individually. @Jackie-Jiang @xiangfu0 this should be reviewable now that the board is clean. -- 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]
