Remove obsolete flag in ShardStateAction$ShardRoutingEntry

jasontedor · jasontedor · commit b58d82f66c8f · 2015-12-03T14:08:35.000-05:00
diff --git a/core/src/main/java/org/elasticsearch/cluster/action/shard/ShardStateAction.java b/core/src/main/java/org/elasticsearch/cluster/action/shard/ShardStateAction.java
@@ -151,7 +151,6 @@ public BatchResult<ShardRoutingEntry> execute(ClusterState currentState, List<Sh
             BatchResult.Builder<ShardRoutingEntry> batchResultBuilder = BatchResult.builder();
             List<FailedRerouteAllocation.FailedShard> shardRoutingsToBeApplied = new ArrayList<>(tasks.size());
             for (ShardRoutingEntry task : tasks) {
-                task.processed = true;
                 shardRoutingsToBeApplied.add(new FailedRerouteAllocation.FailedShard(task.shardRouting, task.message, task.failure));
             }
             ClusterState maybeUpdatedState = currentState;
@@ -201,7 +200,6 @@ public BatchResult<ShardRoutingEntry> execute(ClusterState currentState, List<Sh
             BatchResult.Builder<ShardRoutingEntry> builder = BatchResult.builder();
             List<ShardRouting> shardRoutingsToBeApplied = new ArrayList<>(tasks.size());
             for (ShardRoutingEntry task : tasks) {
-                task.processed = true;
                 shardRoutingsToBeApplied.add(task.shardRouting);
             }
             ClusterState maybeUpdatedState = currentState;
@@ -250,8 +248,6 @@ public static class ShardRoutingEntry extends TransportRequest {
         String message;
         Throwable failure;
 
-        volatile boolean processed; // state field, no need to serialize
-
         public ShardRoutingEntry() {
         }
 
