ctubbsii opened a new pull request #2339:
URL: https://github.com/apache/accumulo/pull/2339
Update Apache parent POM to 24
* Take advantage of enforcer rules built-in to Apache parent POM
* Use minimalJavaBuildVersion and minimalMavenBuildVersion properties
* Remove redundant enforcer checks in our POM
* Remove redundant version information and plugin definitions from
Apache POM that aren't overridden
Update build plugin versions to latest
* Sort BOM dependencies in dependencyManagement before others (a change
in behavior with latest sortpom-maven-plugin that's not possible to
override, but this makes more sense anyway)
* Sort sortpom-maven-plugin's options, ensure blank lines are removed (a
change in the default that is overridable), and ensure space before
the closing slash on empty elements to keep it consistent with other
plugins that update the POM
* Update spotbugs-related Random issues (and minor Random tweaks)
* Always assign SecureRandom objects to SecureRandom variables, so
spotbugs doesn't flag them as insecure Random usages
* Allow SecureRandom objects to be reused using a static final
instance for many classes, to avoid one-off object uses (usually
private, except for a public instance for sharing across ITs)
* While fixing Random-related spotbugs issues, apply naming
consistently
* Remove use of explicit SHA1PRNG implementation of SecureRandom,
preferring non-blocking native implementation (default), and relying
on users to configure their SecureRandom provider through Java
security settings if they want something different from the default
* Remove incorrect attempts to try to seed SecureRandom objects with a
predictable seed (SecureRandom default implementation isn't
predictable, even with a specific seed, since it uses the OS's
native random source)
* Remove unneeded spotbugs warnings suppressions
* Use a fixed-length stream of random numbers in several places where
a loop was used to iterate a fixed number of times and the loop
variable wasn't needed
* For the rare cases where we use a predictable random with a known
seed for testing, pass the seed in the constructor to avoid useless
initiationalization steps to do the initial seed only to
reinitialize immediately with a call to setSeed
Fix pom/license issues related to micrometer (re #2305)
* Move metrics-related dependency versions and transitive dependency
exclusions into project's parent pom's dependencyManagement section
* Update LICENSE to include CC0 artifacts used for metrics dependencies
* Remove property for micrometer version that is only used once for the
micrometer BOM
Other
* Rename incorrect filename RolllingStatsTest.java to
RollingStatsTest.java to match the class name RollingStatsTest
* Update AuthenticationTokenTest to use IntStream.allMatch to loop until
a byte array is generated that isn't all zeros, instead of using
assertFalse, which has a (rare) chance of failing the test
unnecessarily
--
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]
