Avoid allocating the double - remove unused methods

Author: amarziali

utils/queue-utils/src/main/java/datadog/common/queue/BaseQueue.java

@@ -51,7 +51,7 @@ protected BaseQueue(int requestedCapacity) {
   protected BaseQueue(int capacity, int mask) {
     this.capacity = capacity;
     this.mask = mask;
-    this.buffer = (E[]) new Object[capacity << (CACHE_LINE_SHIFT + 1)];
+    this.buffer = (E[]) new Object[capacity << CACHE_LINE_SHIFT];
   }
 
   /**

utils/queue-utils/src/main/java/datadog/common/queue/MpscBlockingConsumerArrayQueueVarHandle.java

@@ -28,15 +28,6 @@ public final class MpscBlockingConsumerArrayQueueVarHandle<E> extends BaseQueue<
   /** The consumer thread that to be woken up if the customer blocked on take() */
   private final PaddedThread blocked = new PaddedThread();
 
-  /**
-   * Store producer limit with release semantics.
-   *
-   * @param value new producer limit
-   */
-  private void soProducerLimit(long value) {
-    producerLimit.setRelease(value);
-  }
-
   /**
    * Load producer limit with opaque semantics (best for hot path). Opaque prevents compiler
    * reordering but has no CPU fence overhead. Safe because racy updates are benign - stale reads
@@ -49,14 +40,12 @@ private long lpProducerLimit() {
   }
 
   /**
-   * Load producer limit with acquire semantics.
+   * Compare and Swap for the producer limit.
    *
-   * @return current producer limit
+   * @param expect
+   * @param update
+   * @return
    */
-  private long laProducerLimit() {
-    return producerLimit.getAcquire();
-  }
-
   private boolean casProducerLimit(long expect, long update) {
     return producerLimit.compareAndSet(expect, update);
   }
@@ -80,15 +69,6 @@ private Thread lpBlocked() {
     return blocked.getOpaque();
   }
 
-  /**
-   * Load blocked thread with acquire semantics.
-   *
-   * @return current blocked thread if any
-   */
-  private Thread laBlocked() {
-    return blocked.getAcquire();
-  }
-
   /**
    * Creates queue with specified capacity.
    *