xiangfu0 commented on PR #18540:
URL: https://github.com/apache/pinot/pull/18540#issuecomment-4765609727
Closing as superseded: `pinot-bom` was added to master by #18542 (merged
2026-05-27). That version lists 61 `pinot-*` artifacts vs the 60 here and
includes `pinot-query-planner-spi`, which this PR is missing — so rebasing this
onto current master would only overwrite the merged BOM with a strict subset.
One thing this PR does that #18542 does not is wire up
`flatten-maven-plugin` (`flattenMode=bom`) to resolve `${project.version}` to a
literal in the *published* POM. If the BOM currently on master ships raw
`${project.version}` and re-interpolates against importers, that's worth a
small standalone follow-up against the existing `pinot-bom` module rather than
reviving this PR. Thanks @anshul98ks123!
--
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]
