Vamsi-klu commented on PR #19082: URL: https://github.com/apache/pinot/pull/19082#issuecomment-5249498075
Apologies for the lag here. My August 6 push said these were coming and did not actually contain them. All four are in `a07aa82` now, with a reply on each thread. The red CI was a separate root cause: three `/** */` doc blocks in `SegmentGenerationJobUtils.java` where master now requires the `///` form, which failed checkstyle and cascaded into Quickstart, Unit Test Set 2 and the compat jobs. Converted in the same push, and the branch is merged with latest master. -- 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]
