Skip to content

Commit b318fdf

Browse files
committed
fix imports ordering and remove matcher that's incompatible with graalvm
1 parent 9d0a32d commit b318fdf

3 files changed

Lines changed: 28 additions & 22 deletions

File tree

google-cloud-jar-parent/pom.xml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,12 +71,6 @@
7171
</dependency>
7272

7373
<!-- Test dependencies -->
74-
<dependency>
75-
<groupId>org.awaitility</groupId>
76-
<artifactId>awaitility</artifactId>
77-
<version>4.3.0</version>
78-
<scope>test</scope>
79-
</dependency>
8074
<dependency>
8175
<groupId>junit</groupId>
8276
<artifactId>junit</artifactId>

java-bigquerystorage/google-cloud-bigquerystorage/src/test/java/com/google/cloud/bigquery/storage/v1beta1/it/ITBigQueryStorageTest.java

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,9 @@
8181
import java.util.Collections;
8282
import java.util.Iterator;
8383
import java.util.List;
84-
import java.util.Objects;
8584
import java.util.UUID;
8685
import java.util.concurrent.TimeUnit;
86+
import java.util.concurrent.atomic.AtomicReference;
8787
import java.util.logging.Logger;
8888
import org.apache.avro.Conversions;
8989
import org.apache.avro.LogicalTypes;
@@ -1233,13 +1233,19 @@ private void ProcessRowsAtSnapshot(
12331233
TableReadOptions.newBuilder().setRowRestriction(filter).build());
12341234
}
12351235

1236-
final CreateReadSessionRequest request = createSessionRequestBuilder.build();
1237-
ReadSession session =
1238-
await()
1239-
.atMost(Duration.ofSeconds(30))
1240-
.pollInterval(Duration.ofSeconds(1))
1241-
.ignoreException(NotFoundException.class)
1242-
.until(() -> client.createReadSession(request), Objects::nonNull);
1236+
CreateReadSessionRequest request = createSessionRequestBuilder.build();
1237+
AtomicReference<ReadSession> sessionRef = new AtomicReference<>();
1238+
await()
1239+
.atMost(Duration.ofSeconds(10))
1240+
.pollInterval(Duration.ofSeconds(1))
1241+
// retry if the newly-created table has not yet fully propagated
1242+
.ignoreException(NotFoundException.class)
1243+
.until(
1244+
() -> {
1245+
sessionRef.set(client.createReadSession(request));
1246+
return true;
1247+
});
1248+
ReadSession session = sessionRef.get();
12431249
assertEquals(
12441250
1,
12451251
session.getStreamsCount(),

java-bigquerystorage/google-cloud-bigquerystorage/src/test/java/com/google/cloud/bigquery/storage/v1beta2/it/ITBigQueryStorageTest.java

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,8 @@
7878
import java.util.Collections;
7979
import java.util.Iterator;
8080
import java.util.List;
81-
import java.util.Objects;
8281
import java.util.UUID;
82+
import java.util.concurrent.atomic.AtomicReference;
8383
import java.util.logging.Logger;
8484
import org.apache.avro.Conversions;
8585
import org.apache.avro.LogicalTypes;
@@ -1214,13 +1214,19 @@ private void ProcessRowsAtSnapshot(
12141214
.setReadOptions(TableReadOptions.newBuilder().setRowRestriction(filter).build());
12151215
}
12161216

1217-
final CreateReadSessionRequest request = createSessionRequestBuilder.build();
1218-
ReadSession session =
1219-
await()
1220-
.atMost(Duration.ofSeconds(30))
1221-
.pollInterval(Duration.ofSeconds(1))
1222-
.ignoreException(NotFoundException.class)
1223-
.until(() -> client.createReadSession(request), Objects::nonNull);
1217+
CreateReadSessionRequest request = createSessionRequestBuilder.build();
1218+
AtomicReference<ReadSession> sessionRef = new AtomicReference<>();
1219+
await()
1220+
.atMost(Duration.ofSeconds(10))
1221+
.pollInterval(Duration.ofSeconds(1))
1222+
// retry if the newly-created table has not yet fully propagated
1223+
.ignoreException(NotFoundException.class)
1224+
.until(
1225+
() -> {
1226+
sessionRef.set(client.createReadSession(request));
1227+
return true;
1228+
});
1229+
ReadSession session = sessionRef.get();
12241230
assertEquals(
12251231
1,
12261232
session.getStreamsCount(),

0 commit comments

Comments
 (0)