Skip to content

Commit eea8b85

Browse files
committed
fix lint
1 parent e9de863 commit eea8b85

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

java-spanner/google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/ChannelFinder.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,8 @@ public final class ChannelFinder {
6363
private final AtomicLong databaseId = new AtomicLong();
6464
private final KeyRecipeCache recipeCache = new KeyRecipeCache();
6565
private final KeyRangeCache rangeCache;
66-
private final Executor cacheUpdateExecutor = MoreExecutors.newSequentialExecutor(CACHE_UPDATE_POOL);
66+
private final Executor cacheUpdateExecutor =
67+
MoreExecutors.newSequentialExecutor(CACHE_UPDATE_POOL);
6768
@Nullable private final EndpointLifecycleManager lifecycleManager;
6869
@Nullable private final String finderKey;
6970

java-spanner/google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/KeyRangeCache.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -558,8 +558,8 @@ private boolean matches(DirectedReadOptions.ReplicaSelection selection) {
558558
* <li>Endpoint exists and READY: usable, do not skip.
559559
* <li>Endpoint exists and TRANSIENT_FAILURE: skip and report in skipped_tablets.
560560
* <li>Endpoint absent, IDLE, CONNECTING, SHUTDOWN, or unsupported: skip silently unless the
561-
* lifecycle manager recently evicted the address for repeated TRANSIENT_FAILURE, in
562-
* which case report it in skipped_tablets.
561+
* lifecycle manager recently evicted the address for repeated TRANSIENT_FAILURE, in which
562+
* case report it in skipped_tablets.
563563
* </ul>
564564
*/
565565
boolean shouldSkip(
@@ -626,8 +626,7 @@ boolean shouldSkip(
626626
return true;
627627
}
628628

629-
private void addSkippedTablet(
630-
RoutingHint.Builder hintBuilder, Set<Long> skippedTabletUids) {
629+
private void addSkippedTablet(RoutingHint.Builder hintBuilder, Set<Long> skippedTabletUids) {
631630
if (!skippedTabletUids.add(tabletUid)) {
632631
return;
633632
}

java-spanner/google-cloud-spanner/src/test/java/com/google/cloud/spanner/spi/v1/EndpointLifecycleManagerTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -282,8 +282,7 @@ public void transientFailureEvictionMarkerRemovedWhenAddressNoLongerActive() thr
282282
awaitCondition(
283283
"endpoint should be evicted after repeated transient failures",
284284
() ->
285-
!manager.isManaged("server1")
286-
&& manager.wasRecentlyEvictedTransientFailure("server1"));
285+
!manager.isManaged("server1") && manager.wasRecentlyEvictedTransientFailure("server1"));
287286

288287
manager.updateActiveAddresses(finder, Collections.emptySet());
289288

0 commit comments

Comments
 (0)