chore(deps): update error-prone monorepo to v2.50.0 (#2212)

This PR contains the following updates:

| Package | Change |
[Age](https://docs.renovatebot.com/merge-confidence/) |
[Confidence](https://docs.renovatebot.com/merge-confidence/) |
|---|---|---|---|
| [com.google.errorprone:error_prone_core](https://errorprone.info)
([source](https://redirect.github.com/google/error-prone)) | `2.49.0` →
`2.50.0` |
![age](https://developer.mend.io/api/mc/badges/age/maven/com.google.errorprone:error_prone_core/2.50.0?slim=true)
|
![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/com.google.errorprone:error_prone_core/2.49.0/2.50.0?slim=true)
|

---

### Release Notes

<details>
<summary>google/error-prone
(com.google.errorprone:error_prone_core)</summary>

###
[`v2.50.0`](https://redirect.github.com/google/error-prone/releases/tag/v2.50.0):
Error Prone 2.50.0

[Compare
Source](https://redirect.github.com/google/error-prone/compare/v2.49.0...v2.50.0)

New checks:

-
[`BoxingComparator`](https://errorprone.info/bugpattern/BoxingComparator):
Detect implicitly-boxing Comparator.comparing key extractors
-
[`ExposedPrivateType`](https://errorprone.info/bugpattern/ExposedPrivateType):
Discourage references to private member classes from non-private APIs
-
[`JUnitMethodInvoked`](https://errorprone.info/bugpattern/JUnitMethodInvoked):
Discourage directly invoking JUnit test methods
-
[`ListRemoveAmbiguous`](https://errorprone.info/bugpattern/ListRemoveAmbiguous):
Detect ambiguous calls to `List.remove(int|Integer)`
-
[`PreferTestParameter`](https://errorprone.info/bugpattern/PreferTestParameter):
suggests using `@TestParameter` instead of `@TestParameters` for
exhaustive boolean and enum parameters on single-element parameterized
tests
-
[`RecordComponentAccessorAnnotationConflict`](https://errorprone.info/bugpattern/RecordComponentAccessorAnnotationConflict):
Detect conflicts between record components and explicit accessor methods
-
[`RecordComponentOverride`](https://errorprone.info/bugpattern/RecordComponentOverride):
Discourage [@&#8203;Override](https://redirect.github.com/Override) on
record component declarations that don't override
  anything
-
[`ThrowableEqualsHashCode`](https://errorprone.info/bugpattern/ThrowableEqualsHashCode):
Discourage overriding `Throwable.equals()` and `hashCode()`

Closed issues:
[#&#8203;5553](https://redirect.github.com/google/error-prone/issues/5553),
[#&#8203;5649](https://redirect.github.com/google/error-prone/issues/5649),
[#&#8203;5778](https://redirect.github.com/google/error-prone/issues/5778)

Full changelog:
<google/error-prone@v2.49.0...v2.50.0>

</details>

---

### Configuration

📅 **Schedule**: (UTC)

- Branch creation
  - At any time (no schedule defined)
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/prometheus/client_java).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4yMDkuNCIsInVwZGF0ZWRJblZlciI6IjQzLjIxOS4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJkZXBlbmRlbmNpZXMiXX0=-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>

Author: renovate[bot]

docs/apidiffs/current_vs_latest/prometheus-metrics-core.txt

@@ -500,7 +500,7 @@
                 <path>
                   <groupId>com.google.errorprone</groupId>
                   <artifactId>error_prone_core</artifactId>
-                  <version>2.49.0</version>
+                  <version>2.50.0</version>
                 </path>
                 <path>
                   <groupId>com.uber.nullaway</groupId>

docs/apidiffs/current_vs_latest/prometheus-metrics-model.txt

@@ -36,6 +36,11 @@
 @StableApi
 public class ExemplarSampler {
 
+  @SuppressWarnings("ReferenceEquality")
+  private static boolean sameObject(Object left, Object right) {
+    return left == right;
+  }
+
   private final ExemplarSamplerConfig config;
   private final Exemplar[] exemplars;
   private final Exemplar[]
@@ -223,7 +228,7 @@ private long doObserveWithoutUpperBounds(double value) {
     int oldestIndex = -1;
     for (int i = 0; i < exemplars.length; i++) {
       Exemplar exemplar = exemplars[i];
-      if (exemplar != null && exemplar != smallest && exemplar != largest) {
+      if (exemplar != null && !sameObject(exemplar, smallest) && !sameObject(exemplar, largest)) {
         if (oldestTimestamp == 0 || exemplar.getTimestampMillis() < oldestTimestamp) {
           oldestTimestamp = exemplar.getTimestampMillis();
           oldestIndex = i;

pom.xml

@@ -34,6 +34,11 @@
  */
 final class CKMSQuantiles {
 
+  @SuppressWarnings("ReferenceEquality")
+  private static boolean sameObject(Object left, Object right) {
+    return left == right;
+  }
+
   final Quantile[] quantiles;
 
   /** Total number of observations (not including those that are still in the buffer). */
@@ -203,7 +208,7 @@ void compress() {
       right = left;
       left = descendingIterator.next();
       r = r - left.g;
-      if (left == samples.getFirst()) {
+      if (sameObject(left, samples.getFirst())) {
         // The min sample must never be merged.
         break;
       }

prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/exemplars/ExemplarSampler.java

@@ -124,7 +124,7 @@ static String[] makePrometheusNames(String[] names) {
     for (int i = 0; i < names.length; i++) {
       String name = names[i];
       if (!PrometheusNaming.isValidLegacyLabelName(name)) {
-        if (prometheusNames == names) {
+        if (sameObject(prometheusNames, names)) {
           prometheusNames = Arrays.copyOf(names, names.length);
         }
         prometheusNames[i] = PrometheusNaming.prometheusName(name);
@@ -234,7 +234,8 @@ public Labels merge(Labels other) {
     }
     String[] names = new String[this.names.length + other.names.length];
     String[] prometheusNames = names;
-    if (this.names != this.prometheusNames || other.names != other.prometheusNames) {
+    if (!sameObject(this.names, this.prometheusNames)
+        || !sameObject(other.names, other.prometheusNames)) {
       prometheusNames = new String[names.length];
     }
     String[] values = new String[names.length];
@@ -244,28 +245,28 @@ public Labels merge(Labels other) {
       if (thisPos >= this.names.length) {
         names[thisPos + otherPos] = other.names[otherPos];
         values[thisPos + otherPos] = other.values[otherPos];
-        if (prometheusNames != names) {
+        if (!sameObject(prometheusNames, names)) {
           prometheusNames[thisPos + otherPos] = other.prometheusNames[otherPos];
         }
         otherPos++;
       } else if (otherPos >= other.names.length) {
         names[thisPos + otherPos] = this.names[thisPos];
         values[thisPos + otherPos] = this.values[thisPos];
-        if (prometheusNames != names) {
+        if (!sameObject(prometheusNames, names)) {
           prometheusNames[thisPos + otherPos] = this.prometheusNames[thisPos];
         }
         thisPos++;
       } else if (this.prometheusNames[thisPos].compareTo(other.prometheusNames[otherPos]) < 0) {
         names[thisPos + otherPos] = this.names[thisPos];
         values[thisPos + otherPos] = this.values[thisPos];
-        if (prometheusNames != names) {
+        if (!sameObject(prometheusNames, names)) {
           prometheusNames[thisPos + otherPos] = this.prometheusNames[thisPos];
         }
         thisPos++;
       } else if (this.prometheusNames[thisPos].compareTo(other.prometheusNames[otherPos]) > 0) {
         names[thisPos + otherPos] = other.names[otherPos];
         values[thisPos + otherPos] = other.values[otherPos];
-        if (prometheusNames != names) {
+        if (!sameObject(prometheusNames, names)) {
           prometheusNames[thisPos + otherPos] = other.prometheusNames[otherPos];
         }
         otherPos++;
@@ -321,6 +322,11 @@ public int compareTo(Labels other) {
   }
 
   // Looks like Java doesn't have a compareTo() method for arrays.
+  @SuppressWarnings("ReferenceEquality")
+  private static boolean sameObject(Object left, Object right) {
+    return left == right;
+  }
+
   private int compare(String[] array1, String[] array2) {
     int result;
     for (int i = 0; i < array1.length; i++) {
@@ -505,14 +511,14 @@ private static void insertionSort(
         int j = i - 1;
         while (j >= left && compare(prometheusNames[j], prometheusName) > 0) {
           names[j + 1] = names[j];
-          if (prometheusNames != names) {
+          if (!sameObject(prometheusNames, names)) {
             prometheusNames[j + 1] = prometheusNames[j];
           }
           values[j + 1] = values[j];
           j--;
         }
         names[j + 1] = name;
-        if (prometheusNames != names) {
+        if (!sameObject(prometheusNames, names)) {
           prometheusNames[j + 1] = prometheusName;
         }
         values[j + 1] = value;
@@ -594,7 +600,7 @@ private static void swap(
       tmp = values[i];
       values[i] = values[j];
       values[j] = tmp;
-      if (prometheusNames != names) {
+      if (!sameObject(prometheusNames, names)) {
         tmp = prometheusNames[i];
         prometheusNames[i] = prometheusNames[j];
         prometheusNames[j] = tmp;

prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/CKMSQuantiles.java

@@ -17,7 +17,7 @@ public class Quantiles implements Iterable<Quantile> {
 
   private Quantiles(List<Quantile> quantiles) {
     quantiles = new ArrayList<>(quantiles);
-    quantiles.sort(Comparator.comparing(Quantile::getQuantile));
+    quantiles.sort(Comparator.comparingDouble(Quantile::getQuantile));
     this.quantiles = Collections.unmodifiableList(quantiles);
     validate();
   }