Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10974 )
Change subject: IMPALA-7311. Allow INSERT on writable partitions even if some other partition is READ_ONLY ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/10974/5/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java File fe/src/main/java/org/apache/impala/catalog/FeFsTable.java: http://gerrit.cloudera.org:8080/#/c/10974/5/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java@470 PS5, Line 470: / No explicit partition was specified. Need to ensure that write access is available : // to all partitions. Why isn't hasWriteAccessToBaseDir needed here? -- To view, visit http://gerrit.cloudera.org:8080/10974 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1dd81100ae73fcabdbfaf679c20cea7dc102cd13 Gerrit-Change-Number: 10974 Gerrit-PatchSet: 5 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tianyi Wang <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]> Gerrit-Comment-Date: Mon, 30 Jul 2018 23:50:50 +0000 Gerrit-HasComments: Yes
