Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15391 )
Change subject: IMPALA-9446 Fix bug that impala failed to read zstd file on s3 ...................................................................... Patch Set 1: Agree with Abhi's comments, I can +2 after addressing. It would be good to run a more complete set of tests against S3 to confirm that the issue is fixed, but I'm OK with merging before that if needed. -- To view, visit http://gerrit.cloudera.org:8080/15391 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I13f8837fda7454ddb4bb47d20a675d6315a3462d Gerrit-Change-Number: 15391 Gerrit-PatchSet: 1 Gerrit-Owner: Xiaomeng Zhang <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Wed, 11 Mar 2020 18:31:12 +0000 Gerrit-HasComments: No
