You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by jm...@apache.org on 2024/02/26 13:28:56 UTC

(accumulo) branch elasticity updated: Remove unused argument to setOperationIdOnce (#4294)

This is an automated email from the ASF dual-hosted git repository.

jmark99 pushed a commit to branch elasticity
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/elasticity by this push:
     new f75dbf8c5b Remove unused argument to setOperationIdOnce (#4294)
f75dbf8c5b is described below

commit f75dbf8c5baa21d2d52ce055f3b627b72a0dc96e
Author: Mark Owens <jm...@apache.org>
AuthorDate: Mon Feb 26 08:28:50 2024 -0500

    Remove unused argument to setOperationIdOnce (#4294)
    
    Remove unused supressError boolean argument from setOperationIdOnce method in TabletMetadata.
    
    Adjust javadoc accordingly.
---
 .../org/apache/accumulo/core/metadata/schema/TabletMetadata.java     | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletMetadata.java b/core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletMetadata.java
index 693ae04b62..5ad11b5945 100644
--- a/core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletMetadata.java
+++ b/core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletMetadata.java
@@ -488,7 +488,7 @@ public class TabletMetadata {
               te.flushNonce = OptionalLong.of(Long.parseUnsignedLong(val, 16));
               break;
             case OPID_QUAL:
-              te.setOperationIdOnce(val, suppressLocationError);
+              te.setOperationIdOnce(val);
               break;
             case SELECTED_QUAL:
               te.selectedFiles = SelectedFiles.from(val);
@@ -581,10 +581,9 @@ public class TabletMetadata {
    * Sets an operation ID only once.
    *
    * @param val operation id to set
-   * @param suppressError set to true to suppress an exception being thrown, else false
    * @throws IllegalStateException if an operation id or location is already set
    */
-  private void setOperationIdOnce(String val, boolean suppressError) {
+  private void setOperationIdOnce(String val) {
     Preconditions.checkState(operationId == null);
     operationId = TabletOperationId.from(val);
   }