michaelsembwever commented on code in PR #2226:
URL: https://github.com/apache/cassandra/pull/2226#discussion_r1140992681
##########
.build/parent-pom-template.xml:
##########
@@ -705,7 +756,7 @@
<dependency>
<groupId>net.openhft</groupId>
<artifactId>chronicle-queue</artifactId>
- <version>5.20.123</version>
Review Comment:
I started off with the bom, then going through maven1 repo used the versions
available (basically aligning to the bom, some patch versions differed).
Then fixing tests some versions needed bumping, posix the example here.
Then on the dev@ thread folk requested we use the latest versions available.
The versions that then got bumped made sense to as they matched sibling
libraries we already had (e.g. jna*), and/or were about newer patch versions
supporting more arch/platforms.
While the initial goal may have been to re-use the tested QA of the bom's
declaration, that's evolved to tailoring it best (with evidence) to our needs
and relying on being in trunk to do our own testing and QA. (IMO this approach
would not be appropriate in one of our release branches.)
--
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]
