Remove nocommit in TransportShardBulkAction

This commit removes a nocommit in TransportShardBulkAction that deserves
a larger issue.

core/src/main/java/org/elasticsearch/action/bulk/TransportShardBulkAction.java

@@ -217,9 +217,6 @@ public class TransportShardBulkAction extends TransportWriteAction<BulkShardRequ
             assert replicaRequest.getPrimaryResponse() != null;
             assert preVersionTypes[requestIndex] != null;
         } catch (Exception e) {
-            // nocommit: since we now have RetryOnPrimaryException, retrying doesn't always mean the shard is closed.
-            // some operations were already perform and have a seqno assigned. we shouldn't just reindex them
-            // add to the meta-issue
             // rethrow the failure if we are going to retry on primary and let parent failure to handle it
             if (retryPrimaryException(e)) {
                 // restore updated versions...