(hbase) branch branch-3 updated: HBASE-29055 Remove the useless parameter from RootProcedureState (#6573)

[pankajkumar]

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

pankajkumar pushed a commit to branch branch-3
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-3 by this push:
     new 8cb628b8c65 HBASE-29055 Remove the useless parameter from 
RootProcedureState (#6573)
8cb628b8c65 is described below

commit 8cb628b8c65a5d051b8b648c1706d7b8228bc499
Author: Peng Lu <lupeng_n...@qq.com>
AuthorDate: Thu Apr 3 02:04:37 2025 +0800

    HBASE-29055 Remove the useless parameter from RootProcedureState (#6573)
    
    Signed-off-by: Pankaj Kumar<pankajku...@apache.org>
    (cherry picked from commit 961f4b12271e6b068a43d610a1e4a0e7888323e7)
---
 .../java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java    | 4 ++--
 .../java/org/apache/hadoop/hbase/procedure2/RootProcedureState.java   | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
 
b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
index f03ccdd1450..b6df5759134 100644
--- 
a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
+++ 
b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureExecutor.java
@@ -1454,7 +1454,7 @@ public class ProcedureExecutor<TEnvironment> {
     }
     do {
       // Try to acquire the execution
-      if (!procStack.acquire(proc)) {
+      if (!procStack.acquire()) {
         if (procStack.setRollback()) {
           // we have the 'rollback-lock' we can start rollingback
           switch (executeRollback(rootProcId, procStack)) {
@@ -1513,7 +1513,7 @@ public class ProcedureExecutor<TEnvironment> {
         default:
           throw new UnsupportedOperationException();
       }
-      procStack.release(proc);
+      procStack.release();
 
       if (proc.isSuccess()) {
         // update metrics on finishing the procedure
diff --git 
a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/RootProcedureState.java
 
b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/RootProcedureState.java
index c9f5bad2a13..b9d9f7f5626 100644
--- 
a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/RootProcedureState.java
+++ 
b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/RootProcedureState.java
@@ -135,7 +135,7 @@ class RootProcedureState<TEnvironment> {
   /**
    * Called by the ProcedureExecutor to mark the procedure step as running.
    */
-  protected synchronized boolean acquire(Procedure<TEnvironment> proc) {
+  protected synchronized boolean acquire() {
     if (state != State.RUNNING) {
       return false;
     }
@@ -147,7 +147,7 @@ class RootProcedureState<TEnvironment> {
   /**
    * Called by the ProcedureExecutor to mark the procedure step as finished.
    */
-  protected synchronized void release(Procedure<TEnvironment> proc) {
+  protected synchronized void release() {
     running--;
   }