hip 1299 - switch error from INVALID_NODE_ACCOUNT_ID to INVALID_NODE_ACCOUNT

Signed-off-by: emiliyank <e.kadiyski@gmail.com>

Author: emiliyank

sdk/src/main/java/com/hedera/hashgraph/sdk/Executable.java

@@ -442,10 +442,10 @@ public O execute(Client client, Duration timeout) throws TimeoutException, Prech
                     lastException = grpcRequest.mapStatusException();
                     advanceRequest(); // Advance to next node before retrying
 
-                    // Handle INVALID_NODE_ACCOUNT_ID after advancing (matches Go SDK's executionStateRetryWithAnotherNode)
-                    if (status == Status.INVALID_NODE_ACCOUNT_ID) {
+                    // Handle INVALID_NODE_ACCOUNT after advancing (matches Go SDK's executionStateRetryWithAnotherNode)
+                    if (status == Status.INVALID_NODE_ACCOUNT) {
                         logger.trace(
-                                "Received INVALID_NODE_ACCOUNT_ID; updating address book and marking node {} as unhealthy, attempt #{}",
+                                "Received INVALID_NODE_ACCOUNT; updating address book and marking node {} as unhealthy, attempt #{}",
                                 node.getAccountId(),
                                 attempt);
                         // Schedule async address book update (matches Go's defer client._UpdateAddressBook())
@@ -790,10 +790,10 @@ private void executeAsyncInternal(
                                         case SERVER_ERROR:
                                             advanceRequest(); // Advance to next node before retrying
 
-                                            // Handle INVALID_NODE_ACCOUNT_ID after advancing (matches Go SDK's executionStateRetryWithAnotherNode)
-                                            if (status == Status.INVALID_NODE_ACCOUNT_ID) {
+                                            // Handle INVALID_NODE_ACCOUNT after advancing (matches Go SDK's executionStateRetryWithAnotherNode)
+                                            if (status == Status.INVALID_NODE_ACCOUNT) {
                                                 logger.trace(
-                                                        "Received INVALID_NODE_ACCOUNT_ID; updating address book and marking node {} as unhealthy, attempt #{}",
+                                                        "Received INVALID_NODE_ACCOUNT; updating address book and marking node {} as unhealthy, attempt #{}",
                                                         grpcRequest.getNode().getAccountId(),
                                                         attempt);
                                                 // Schedule async address book update (matches Go's defer client._UpdateAddressBook())
@@ -901,9 +901,10 @@ ExecutionState getExecutionState(Status status, ResponseT response) {
                 return ExecutionState.SERVER_ERROR;
             case BUSY:
                 return ExecutionState.RETRY;
-            case INVALID_NODE_ACCOUNT_ID:
+            case INVALID_NODE_ACCOUNT:
                 // Matches Go SDK's executionStateRetryWithAnotherNode behavior:
                 // immediately retry with next node without delay
+                // This occurs when a node's account ID has changed
                 return ExecutionState.SERVER_ERROR;
             case OK:
                 return ExecutionState.SUCCESS;
@@ -1010,9 +1011,9 @@ O mapResponse() {
         }
 
         void handleResponse(ResponseT response, Status status, ExecutionState executionState, @Nullable Client client) {
-            // Note: For INVALID_NODE_ACCOUNT_ID, we don't mark the node as unhealthy here
+            // Note: For INVALID_NODE_ACCOUNT, we don't mark the node as unhealthy here
             // because we need to do it AFTER advancing the request, to match Go SDK behavior
-            if (status != Status.INVALID_NODE_ACCOUNT_ID) {
+            if (status != Status.INVALID_NODE_ACCOUNT) {
                 node.decreaseBackoff();
             }
 
@@ -1042,9 +1043,9 @@ void handleResponse(ResponseT response, Status status, ExecutionState executionS
                     verboseLog(node);
                 }
                 case SERVER_ERROR -> {
-                    // Note: INVALID_NODE_ACCOUNT_ID is handled after advanceRequest() in execute methods
+                    // Note: INVALID_NODE_ACCOUNT is handled after advanceRequest() in execute methods
                     // to match Go SDK's executionStateRetryWithAnotherNode behavior
-                    if (status != Status.INVALID_NODE_ACCOUNT_ID) {
+                    if (status != Status.INVALID_NODE_ACCOUNT) {
                         logger.warn(
                                 "Problem submitting request to node {} for attempt #{}, retry with new node: {}",
                                 node.getAccountId(),

sdk/src/test/java/com/hedera/hashgraph/sdk/ExecutableTest.java

@@ -512,9 +512,9 @@ void shouldRetryReturnsCorrectStates() {
                 .isEqualTo(ExecutionState.SERVER_ERROR);
         assertThat(tx.getExecutionState(Status.PLATFORM_NOT_ACTIVE, null)).isEqualTo(ExecutionState.SERVER_ERROR);
         assertThat(tx.getExecutionState(Status.BUSY, null)).isEqualTo(ExecutionState.RETRY);
-        // INVALID_NODE_ACCOUNT_ID now returns SERVER_ERROR to match Go SDK's executionStateRetryWithAnotherNode
+        // INVALID_NODE_ACCOUNT now returns SERVER_ERROR to match Go SDK's executionStateRetryWithAnotherNode
         // which immediately retries with another node without delay
-        assertThat(tx.getExecutionState(Status.INVALID_NODE_ACCOUNT_ID, null)).isEqualTo(ExecutionState.SERVER_ERROR);
+        assertThat(tx.getExecutionState(Status.INVALID_NODE_ACCOUNT, null)).isEqualTo(ExecutionState.SERVER_ERROR);
         assertThat(tx.getExecutionState(Status.OK, null)).isEqualTo(ExecutionState.SUCCESS);
         assertThat(tx.getExecutionState(Status.ACCOUNT_DELETED, null)).isEqualTo(ExecutionState.REQUEST_ERROR);
     }

sdk/src/testIntegration/java/com/hedera/hashgraph/sdk/test/integration/NodeUpdateAccountIdIntegrationTest.java

@@ -127,9 +127,9 @@ void testNodeUpdateTransactionCanChangeNodeAccountUpdateAddressbookAndRetry() th
             var newAccountKey = PrivateKey.generateED25519();
 
             // Submit to node 3 and node 4
-            // Node 3 will fail with INVALID_NODE_ACCOUNT_ID (because it now uses newNodeAccountID)
+            // Node 3 will fail with INVALID_NODE_ACCOUNT (because it now uses newNodeAccountID)
             // The SDK should automatically:
-            // 1. Detect INVALID_NODE_ACCOUNT_ID error
+            // 1. Detect INVALID_NODE_ACCOUNT error
             // 2. Advance to next node
             // 3. Update the address book asynchronously
             // 4. Mark node 3 as unhealthy