abdullah alamoudi has submitted this change and it was merged. Change subject: Fix Asterix Stream Reader ......................................................................
Fix Asterix Stream Reader The reset method in the asterix stream reader wasn't working. Calling clear() on a byte buffer only reset the position but not the limit. hence, the reader still thinks that there are some leftover data. This has been fixed to set the limit to 0 Change-Id: I41a84baacdd233120ca08eeebbd2d757b4e5c154 Reviewed-on: https://asterix-gerrit.ics.uci.edu/761 Reviewed-by: Yingyi Bu <[email protected]> Tested-by: Jenkins <[email protected]> --- M .gitignore M asterix-external-data/src/main/java/org/apache/asterix/external/input/stream/AsterixInputStreamReader.java 2 files changed, 2 insertions(+), 1 deletion(-) Approvals: Yingyi Bu: Looks good to me, approved Jenkins: Verified diff --git a/.gitignore b/.gitignore index 8d0e894..926dcfe 100644 --- a/.gitignore +++ b/.gitignore @@ -14,6 +14,7 @@ asterix-app/mdtest/ asterix-app/opttest/ asterix-app/parserts/ +asterix-app/library/ asterix-app/opt_parserts/ asterix-app/runtime_parserts/ asterix-app/data/csv/beer.csv diff --git a/asterix-external-data/src/main/java/org/apache/asterix/external/input/stream/AsterixInputStreamReader.java b/asterix-external-data/src/main/java/org/apache/asterix/external/input/stream/AsterixInputStreamReader.java index b1ef892..7e280a5 100644 --- a/asterix-external-data/src/main/java/org/apache/asterix/external/input/stream/AsterixInputStreamReader.java +++ b/asterix-external-data/src/main/java/org/apache/asterix/external/input/stream/AsterixInputStreamReader.java @@ -115,6 +115,6 @@ @Override public void reset() throws IOException { - byteBuffer.clear(); + byteBuffer.limit(0); } } -- To view, visit https://asterix-gerrit.ics.uci.edu/761 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41a84baacdd233120ca08eeebbd2d757b4e5c154 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]>
