zentol commented on a change in pull request #6782: [FLINK-9083][Cassandra
Connector] Add async backpressure support to Cassandra Connector
URL: https://github.com/apache/flink/pull/6782#discussion_r235144945
##########
File path:
flink-connectors/flink-connector-cassandra/src/test/java/org/apache/flink/streaming/connectors/cassandra/CassandraSinkBaseTest.java
##########
@@ -84,111 +90,108 @@ public void testThrowErrorOnClose() throws Exception {
casSinkFunc.open(new Configuration());
Exception cause = new RuntimeException();
-
casSinkFunc.setResultFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
+
casSinkFunc.enqueueResultSetFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
casSinkFunc.invoke("hello");
try {
casSinkFunc.close();
Assert.fail("Close should have thrown an exception.");
} catch (IOException e) {
Assert.assertEquals(cause, e.getCause());
- Assert.assertEquals(0,
casSinkFunc.getNumOfPendingRecords());
+ Assert.assertEquals(0,
casSinkFunc.getAcquiredPermits());
}
}
@Test(timeout = 5000)
public void testThrowErrorOnInvoke() throws Exception {
- TestCassandraSink casSinkFunc = new TestCassandraSink();
+ try (TestCassandraSink casSinkFunc = new TestCassandraSink()) {
+ casSinkFunc.open(new Configuration());
- casSinkFunc.open(new Configuration());
-
- Exception cause = new RuntimeException();
-
casSinkFunc.setResultFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
+ Exception cause = new RuntimeException();
+
casSinkFunc.enqueueResultSetFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
- casSinkFunc.invoke("hello");
+ casSinkFunc.invoke("hello");
- try {
- casSinkFunc.invoke("world");
- Assert.fail("Sending of second value should have
failed.");
- } catch (IOException e) {
- Assert.assertEquals(cause, e.getCause());
- Assert.assertEquals(0,
casSinkFunc.getNumOfPendingRecords());
+ try {
+ casSinkFunc.invoke("world");
+ Assert.fail("Sending of second value should
have failed.");
+ } catch (IOException e) {
+ Assert.assertEquals(cause, e.getCause());
+ Assert.assertEquals(0,
casSinkFunc.getAcquiredPermits());
+ }
}
}
@Test(timeout = 5000)
public void testIgnoreError() throws Exception {
Exception cause = new RuntimeException();
CassandraFailureHandler failureHandler = failure ->
Assert.assertEquals(cause, failure);
- TestCassandraSink casSinkFunc = new
TestCassandraSink(failureHandler);
+ try (TestCassandraSink casSinkFunc = new
TestCassandraSink(failureHandler)) {
+ casSinkFunc.open(new Configuration());
- casSinkFunc.open(new Configuration());
+
casSinkFunc.enqueueResultSetFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
+
casSinkFunc.enqueueResultSetFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
-
casSinkFunc.setResultFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
-
- casSinkFunc.invoke("hello");
- casSinkFunc.invoke("world");
+ casSinkFunc.invoke("hello");
+ casSinkFunc.invoke("world");
+ }
}
@Test(timeout = 5000)
public void testThrowErrorOnSnapshot() throws Exception {
TestCassandraSink casSinkFunc = new TestCassandraSink();
- OneInputStreamOperatorTestHarness<String, Object> testHarness =
- new OneInputStreamOperatorTestHarness<>(new
StreamSink<>(casSinkFunc));
-
- testHarness.open();
+ try (OneInputStreamOperatorTestHarness<String, Object>
testHarness =
+ new OneInputStreamOperatorTestHarness<>(new
StreamSink<>(casSinkFunc))) {
+ testHarness.open();
- Exception cause = new RuntimeException();
-
casSinkFunc.setResultFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
+ Exception cause = new RuntimeException();
+
casSinkFunc.enqueueResultSetFuture(ResultSetFutures.fromCompletableFuture(FutureUtils.getFailedFuture(cause)));
- casSinkFunc.invoke("hello");
+ casSinkFunc.invoke("hello");
- try {
- testHarness.snapshot(123L, 123L);
+ try {
Review comment:
I would appreciate if we would move the majority of these changes into a
separate commit. (try-catch introduction mostly).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
