dependabot[bot] opened a new pull request, #20048: URL: https://github.com/apache/pinot/pull/20048
Bumps [com.google.errorprone:error_prone_annotations](https://github.com/google/error-prone) from 2.50.0 to 2.51.0. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/google/error-prone/releases";>com.google.errorprone:error_prone_annotations's releases</a>.</em></p> <blockquote> <h2>Error Prone 2.51.0</h2> <p>New checks:</p> <ul> <li><a href="https://errorprone.info/bugpattern/HashCodeInObjectsHash";><code>HashCodeInObjectsHash</code></a>: Detect redundant <code>hashCode()</code> or <code>Objects.hash()</code> calls inside <code>Objects.hash(...)</code>, and simplify single-argument <code>Objects.hash(x)</code> calls to <code>Objects.hashCode(x)</code>.</li> <li><a href="https://errorprone.info/bugpattern/LoopToTestParameter";><code>LoopToTestParameter</code></a>: Suggest migrating loops in unit tests to <code>TestParameterInjector</code>.</li> <li><a href="https://errorprone.info/bugpattern/MissingJavadoc";><code>MissingJavadoc</code></a>: Flag public types that are missing Javadoc comments (disabled by default).</li> <li><a href="https://errorprone.info/bugpattern/PreferCharsetOverload";><code>PreferCharsetOverload</code></a>: Prefer calling method and constructor overloads that accept a <code>Charset</code> over those that accept a <code>String</code> encoding name.</li> <li><a href="https://errorprone.info/bugpattern/PreferPreconditions";><code>PreferPreconditions</code></a>: Suggest using Guava's <code>Preconditions</code> instead of explicit <code>if</code>-<code>throw</code> statements for parameter validation (disabled by default).</li> <li><a href="https://errorprone.info/bugpattern/RequireNonNullRefactoring";><code>RequireNonNullRefactoring</code></a>: Refactor explicit <code>if (x == null) { throw new NullPointerException(...); }</code> checks to <code>Objects.requireNonNull</code> (disabled by default).</li> <li><a href="https://errorprone.info/bugpattern/SimplifyTestParameter";><code>SimplifyTestParameter</code></a>: Remove redundant explicit value lists on <code>@TestParameter</code> annotations for <code>boolean</code> and <code>Enum</code> types when all values are specified (disabled by default).</li> <li><a href="https://errorprone.info/bugpattern/TraditionalJavadocToMarkdown";><code>TraditionalJavadocToMarkdown</code></a>: Convert traditional <code>/** ... */</code> Javadoc comments into <code>///</code> Markdown Javadoc comments (disabled by default).</li> <li><a href="https://errorprone.info/bugpattern/ValueClassIdentity";><code>ValueClassIdentity</code></a>: Flag reliance on identity semantics (such as synchronization, <code>WeakReference</code>, <code>IdentityHashMap</code>, or <code>System.identityHashCode</code>) of value-based classes (disabled by default).</li> </ul> <p>Other changes:</p> <ul> <li><a href="https://errorprone.info/bugpattern/CompareToZero";><code>CompareToZero</code></a>: Upgraded from warning to error, and expanded to flag non-relational operations, unary negation, and <code>switch</code> statements/expressions on <code>compareTo</code> and <code>compare</code> results without <code>Integer.signum</code>.</li> <li><a href="https://errorprone.info/bugpattern/IfChainToSwitch";><code>IfChainToSwitch</code></a>: Enabled by default as a warning.</li> <li><a href="https://errorprone.info/bugpattern/UndefinedEquals";><code>UndefinedEquals</code></a>: Merged <code>ArrayAsKeyOfSetOrMap</code> and <code>CollectionUndefinedEquality</code> into <code>UndefinedEquals</code> (retaining their names as <code>altNames</code>), and generalized detection of types without well-defined <code>equals</code> or <code>hashCode</code> across direct equality, collection containment, and hash-based containers.</li> <li><a href="https://errorprone.info/bugpattern/EqualsIncompatibleType";><code>EqualsIncompatibleType</code></a> / <a href="https://errorprone.info/bugpattern/TruthIncompatibleType";><code>TruthIncompatibleType</code></a>: Flag equality comparisons between distinct <code>record</code> types (<a href="https://redirect.github.com/google/error-prone/issues/6107";>#6107</a>).</li> <li><a href="https://errorprone.info/bugpattern/ReferenceEquality";><code>ReferenceEquality</code></a>: Allow <code>==</code> comparisons on <code>private</code> classes whose type hierarchy does not override <code>Object.equals</code> (<a href="https://redirect.github.com/google/error-prone/issues/5865";>#5865</a>), <code>private static final</code> sentinel fields, <code>Thread</code> (<a href="https://redirect.github.com/google/error-prone/issues/5900";>#5900</a>), and <code>com.sun.source.tree.Tree</code>.</li> <li><a href="https://errorprone.info/bugpattern/PatternMatchingInstanceof";><code>PatternMatchingInstanceof</code></a>: Support <code>while</code> and <code>for</code> loops, parenthesized expressions (<a href="https://redirect.github.com/google/error-prone/issues/5843";>#5843</a>), and replacing additional redundant casts when the <code>instanceof</code> already introduces a pattern variable.</li> <li><a href="https://errorprone.info/bugpattern/DistinctVarargsChecker";><code>DistinctVarargsChecker</code></a>: Expanded to check <code>Objects.hash</code>, <code>CompletableFuture.allOf</code>/<code>anyOf</code>, <code>EnumSet.of</code>, <code>Sets.union</code>/<code>intersection</code>/<code>difference</code>, <code>Stream.concat</code>, <code>Math.min</code>/<code>max</code>, and additional JDK, Guava, Protobuf, Truth, and Mockito methods.</li> <li><a href="https://errorprone.info/bugpattern/MustBeClosedChecker";><code>MustBeClosedChecker</code></a>: Require <code>java.lang.foreign.Arena.ofConfined</code> and <code>Arena.ofShared</code> to be closed.</li> <li><a href="https://errorprone.info/bugpattern/ImpossibleNullComparison";><code>ImpossibleNullComparison</code></a>: Flag null checks on the return value of <code>Class.forName</code>.</li> <li><a href="https://errorprone.info/bugpattern/RestrictedApi";><code>RestrictedApi</code></a> / <a href="https://errorprone.info/bugpattern/RestrictedInheritance";><code>RestrictedInheritance</code></a>: Added an <code>allowedPaths</code> attribute to allow calls or inheritance from matching file paths without a regular expression.</li> <li><a href="https://errorprone.info/bugpattern/InlineMeInliner";><code>InlineMeInliner</code></a>: Support substituting type arguments at call sites when inlining generic <code>@InlineMe</code> methods.</li> <li><a href="https://errorprone.info/bugpattern/WellKnownKeep";><code>WellKnownKeep</code></a>: Recognize Apache Beam <code>DoFn</code> annotations and retain open-source library annotations (<code>Gson</code>, <code>Mockito</code>, <code>Guice</code>, <code>Objectify</code>, <code>TestParameterInjector</code>, <code>AutoFactory</code>).</li> <li>Per-check timing collection is now disabled by default, reducing Error Prone compile-time overhead by ~3–5%; pass <code>-XepPrintTimings</code> to print a timing report after compilation or <code>-XepRecordTimings</code> to record timings programmatically (<a href="https://redirect.github.com/google/error-prone/issues/6079";>#6079</a>).</li> <li><code>CompilationTestHelper</code> now reports all lines with mismatched diagnostic expectations in a single test run rather than failing on the first mismatch (<a href="https://redirect.github.com/google/error-prone/issues/6116";>#6116</a>).</li> <li>Replaced the RE2/J dependency with <a href="https://github.com/eaftan/safere";>SafeRE</a>.</li> </ul> <p>Closed issues: <a href="https://redirect.github.com/google/error-prone/issues/4914";>#4914</a>, <a href="https://redirect.github.com/google/error-prone/issues/5182";>#5182</a>, <a href="https://redirect.github.com/google/error-prone/issues/5843";>#5843</a>, <a href="https://redirect.github.com/google/error-prone/issues/5857";>#5857</a>, <a href="https://redirect.github.com/google/error-prone/issues/5865";>#5865</a>, <a href="https://redirect.github.com/google/error-prone/issues/5878";>#5878</a>, <a href="https://redirect.github.com/google/error-prone/issues/5900";>#5900</a>, <a href="https://redirect.github.com/google/error-prone/issues/6029";>#6029</a>, <a href="https://redirect.github.com/google/error-prone/issues/6079";>#6079</a>, <a href="https://redirect.github.com/google/error-prone/issues/6107";>#6107</a>, <a href="https://redirect.github.com/google/error-prone/issues/6116";>#6116</a>, <a href="https://redirect.github.com/google/error-prone/issues/6133";>#6133</a></p> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/google/error-prone/commit/3e9c316c3e3f4dc8b3a2011701a6488e1c23e680";><code>3e9c316</code></a> Release Error Prone 2.51.0</li> <li><a href="https://github.com/google/error-prone/commit/4f83ec377cc37c4a0d0967779aa422c9f6fe124a";><code>4f83ec3</code></a> Add Maven 3.9.16 setup to release workflow</li> <li><a href="https://github.com/google/error-prone/commit/47eb5be1fe42c30d73b62cf0af272488ffbdebe1";><code>47eb5be</code></a> Exempt Apache Beam DoFn method annotations in WellKnownKeep</li> <li><a href="https://github.com/google/error-prone/commit/7e2efead12b4c099310fc5d7bb3456e6bd834ead";><code>7e2efea</code></a> [IfChainToSwitch] Lower the minimum if-chain length from <code>5</code> to <code>4</code></li> <li><a href="https://github.com/google/error-prone/commit/0a9708634f5683865cfc30e08c1ebaf694c8c0ba";><code>0a97086</code></a> [IfChainToSwitch] address issue where deleting trailing code can cause compil...</li> <li><a href="https://github.com/google/error-prone/commit/f61d405aafd46662fbb5a69374f5333197b55e22";><code>f61d405</code></a> [Reachability] drop <code>skipLabel</code> from <code>visitContinue</code></li> <li><a href="https://github.com/google/error-prone/commit/854665b6924534eb866aa5cf5e65a3e42a506718";><code>854665b</code></a> Collapse duplicated package prefixes in <code>_javac</code> source jar paths.</li> <li><a href="https://github.com/google/error-prone/commit/35884331592899a49c692bd5d0b3aca518869f34";><code>3588433</code></a> Introduce <code>-XepOpt:Nullness:LikelyPublicApiOnly=true</code> for annotating only API...</li> <li><a href="https://github.com/google/error-prone/commit/b8a8390ac9938b9ec59530166525f0ef01f0b78d";><code>b8a8390</code></a> Merge CollectionUndefinedEquality and ArrayAsKeyOfSetOrMap into UndefinedEquals.</li> <li><a href="https://github.com/google/error-prone/commit/59f98c3d9c0cb42a82f93a2661a54ba98d40803f";><code>59f98c3</code></a> Allow <code>==</code> comparisons on <code>Tree</code> instances.</li> <li>Additional commits viewable in <a href="https://github.com/google/error-prone/compare/v2.50.0...v2.51.0";>compare view</a></li> </ul> </details> <br /> [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> -- 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]
