Taras Bobrovytsky has posted comments on this change. ( http://gerrit.cloudera.org:8080/10646 )
Change subject: IMPALA-7102: Disable support of erasure coding by default ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/10646/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/10646/1//COMMIT_MSG@9 PS1, Line 9: In this patch we add a query option ALLOW_ERASURE_CODED_FILES, that > I had a high-level question: what's the rationale for making it a query opt Updated the commit message. The feature should work in principle and has been tested to a certain extent already and it works. It wouldn't hurt to make it possible for advanced users to be able to enabled the feature if they know what they are doing. http://gerrit.cloudera.org:8080/#/c/10646/1/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java File fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java: http://gerrit.cloudera.org:8080/#/c/10646/1/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java@786 PS1, Line 786: "The query involves scanning an erasure coded file %s located in %s. " + Updated the message -- To view, visit http://gerrit.cloudera.org:8080/10646 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icd3b1754541262467a6e67068b0b447882a40fb3 Gerrit-Change-Number: 10646 Gerrit-PatchSet: 1 Gerrit-Owner: Taras Bobrovytsky <[email protected]> Gerrit-Reviewer: Adrian Ng Gerrit-Reviewer: Taras Bobrovytsky <[email protected]> Gerrit-Reviewer: Tianyi Wang <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Wed, 27 Jun 2018 00:01:01 +0000 Gerrit-HasComments: Yes
