abdullah alamoudi has submitted this change and it was merged. Change subject: Fix Input Handler Test ......................................................................
Fix Input Handler Test The test was checking that the data was fully consumed before it closes the input handler. The test was fixed to close it before it checks. Change-Id: I6e0e8f130a8250d9cddeb9077f9c4b79491f6fd9 Reviewed-on: https://asterix-gerrit.ics.uci.edu/872 Reviewed-by: Michael Blow <[email protected]> Reviewed-by: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> --- M asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/feed/test/InputHandlerTest.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Michael Blow: Looks good to me, approved Jenkins: Looks good to me, but someone else must approve; Verified diff --git a/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/feed/test/InputHandlerTest.java b/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/feed/test/InputHandlerTest.java index 705d5e3..1312721 100644 --- a/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/feed/test/InputHandlerTest.java +++ b/asterixdb/asterix-external-data/src/test/java/org/apache/asterix/external/feed/test/InputHandlerTest.java @@ -714,6 +714,7 @@ Assert.assertEquals(1, handler.framesOnDisk()); writer.unfreeze(); result.get(); + handler.close(); Assert.assertEquals(0, handler.framesOnDisk()); } catch (Throwable th) { th.printStackTrace(); -- To view, visit https://asterix-gerrit.ics.uci.edu/872 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e0e8f130a8250d9cddeb9077f9c4b79491f6fd9 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]>
