Committed by
Gerrit Code Review
Minor touchups
Change-Id: I0fee1becb6cbb002b30d45a06e7ea3523636dfb4
Showing
2 changed files
with
0 additions
and
6 deletions
| ... | @@ -158,7 +158,6 @@ public class AsyncConsistentSetMultimap | ... | @@ -158,7 +158,6 @@ public class AsyncConsistentSetMultimap |
| 158 | 158 | ||
| 159 | @Override | 159 | @Override |
| 160 | public CompletableFuture<Map<String, Collection<byte[]>>> asMap() { | 160 | public CompletableFuture<Map<String, Collection<byte[]>>> asMap() { |
| 161 | - //TODO | ||
| 162 | throw new UnsupportedOperationException("Expensive operation."); | 161 | throw new UnsupportedOperationException("Expensive operation."); |
| 163 | } | 162 | } |
| 164 | 163 | ... | ... |
| ... | @@ -476,11 +476,6 @@ public class AsyncConsistentSetMultimapState extends ResourceStateMachine | ... | @@ -476,11 +476,6 @@ public class AsyncConsistentSetMultimapState extends ResourceStateMachine |
| 476 | }); | 476 | }); |
| 477 | if (valuesToAdd.isEmpty()) { | 477 | if (valuesToAdd.isEmpty()) { |
| 478 | //Do not increment or add the commit if no change resulted | 478 | //Do not increment or add the commit if no change resulted |
| 479 | -// TODO fairly sure the below case is unreachable but | ||
| 480 | -// TODO need to make sure | ||
| 481 | -// if (valueCountdownMap.isEmpty()) { | ||
| 482 | -// backingMap.remove(((Put) commit.operation()).key()); | ||
| 483 | -// } | ||
| 484 | commit.close(); | 479 | commit.close(); |
| 485 | return null; | 480 | return null; |
| 486 | } | 481 | } | ... | ... |
-
Please register or login to post a comment