Alex Behm has posted comments on this change. Change subject: IMPALA-1702: Enforce table level consistency accross service ......................................................................
Patch Set 7: (6 comments) http://gerrit.cloudera.org:8080/#/c/4349/7/fe/src/main/java/org/apache/impala/analysis/DescriptorTable.java File fe/src/main/java/org/apache/impala/analysis/DescriptorTable.java: Line 154: public TDescriptorTable toThrift() { Let's add a comment about table id assignment and that this also checks the single-table consistency guarantee. Line 156: // Maps from base table to its table id used in backend. in the backend http://gerrit.cloudera.org:8080/#/c/4349/7/fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java File fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java: Line 42: import com.google.common.base.Preconditions; remove http://gerrit.cloudera.org:8080/#/c/4349/7/fe/src/main/java/org/apache/impala/service/Frontend.java File fe/src/main/java/org/apache/impala/service/Frontend.java: Line 1128: * 1. All tables sent to backend have a unique Id, assigned in Let's rework/move this validation as follows: * Let's check the table-level consistency guarantee directly in DescriptorTbl.toThrift(). * Let's add validateTableIds() to PlannerTestBase that checks the uniqueness of table ids in the produced TDescriptorTable * Get rid of this current function and remove references to IMPALA-1702. Line 1134: private void validateTableIds(Analyzer analyzer, TExecRequest result) validateTableConsistency()? http://gerrit.cloudera.org:8080/#/c/4349/7/fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java File fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java: Line 93: private final Map<Long, TTableDescriptor> tableMap_ = Maps.newHashMap(); Integer -- To view, visit http://gerrit.cloudera.org:8080/4349 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifad648b72684ae495ec387590ab1bc58ce5b39e2 Gerrit-PatchSet: 7 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Huaisi Xu <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Huaisi Xu <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-HasComments: Yes
