You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "David Ribeiro Alves (Code Review)" <ge...@cloudera.org> on 2016/03/14 07:56:48 UTC

[Impala-CR](cdh5-trunk) Review Only: Final Impala-Kudu merge

David Ribeiro Alves has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2536

Change subject: Review Only: Final Impala-Kudu merge
......................................................................

Review Only: Final Impala-Kudu merge

This is a review-only version of the mostly mechanical merge between 'feature/kudu' and 'cdh5-trunk'.
The following non-mechanical changes were made:

be/src/exec/CMakeLists.txt - Was changed to disable -Wno-error when compiling certain kudu files (there is a macro clash between gutil and kudu's stubs.h that is generating a warning and thus making the build fail. This is a temporary workaround, filed IMPALA-3182)
be/src/exec/kudu-scan-node.h - Along with the .cc file, these were changed to allow to store the callback id when using AddThreadAvailableCb()
be/src/exec/kudu-scanner.cc - Changed the logic in RelocateValuesFromKudu() to account for the fact that MemPool::Allocate() no longer returns NULL if asked to allocate a 0 size memory region.
be/src/exec/kudu-table-sink.h - Along with the .cc file, these were changed to support the new FlushFinal(RuntimeState* state) method
fe/src/main/java/com/cloudera/impala/analysis/SelectStmt.java - registerIsNotEmptyPredicates(Analyzer analyzer) was changed to account for the new FromClause

Change-Id: Ifa7d84ef583fc1d0ea2b656e3c0f41e9775a4659
---
M CMakeLists.txt
M be/CMakeLists.txt
M be/src/common/init.cc
M be/src/common/names.h
M be/src/common/status.h
M be/src/exec/CMakeLists.txt
M be/src/exec/data-sink.cc
M be/src/exec/exec-node.cc
M be/src/exec/exec-node.h
A be/src/exec/kudu-scan-node-test.cc
A be/src/exec/kudu-scan-node.cc
A be/src/exec/kudu-scan-node.h
A be/src/exec/kudu-scanner.cc
A be/src/exec/kudu-scanner.h
A be/src/exec/kudu-table-sink-test.cc
A be/src/exec/kudu-table-sink.cc
A be/src/exec/kudu-table-sink.h
A be/src/exec/kudu-testutil.h
A be/src/exec/kudu-util.cc
A be/src/exec/kudu-util.h
M be/src/rpc/authentication-test.cc
M be/src/runtime/coordinator.cc
M be/src/runtime/descriptors.cc
M be/src/runtime/descriptors.h
M be/src/scheduling/simple-scheduler.cc
M be/src/testutil/desc-tbl-builder.cc
M be/src/testutil/desc-tbl-builder.h
A be/src/testutil/test-macros.h
M be/src/util/blocking-queue-test.cc
M be/src/util/blocking-queue.h
M bin/bootstrap_toolchain.py
M bin/clean.sh
M bin/impala-config.sh
M common/thrift/CatalogObjects.thrift
M common/thrift/DataSinks.thrift
M common/thrift/Descriptors.thrift
M common/thrift/ImpalaInternalService.thrift
M common/thrift/JniCatalog.thrift
M common/thrift/PlanNodes.thrift
M common/thrift/generate_error_codes.py
M fe/pom.xml
M fe/src/main/cup/sql-parser.cup
M fe/src/main/java/com/cloudera/impala/analysis/AlterTableStmt.java
M fe/src/main/java/com/cloudera/impala/analysis/AnalysisContext.java
M fe/src/main/java/com/cloudera/impala/analysis/Analyzer.java
M fe/src/main/java/com/cloudera/impala/analysis/BaseTableRef.java
M fe/src/main/java/com/cloudera/impala/analysis/BinaryPredicate.java
M fe/src/main/java/com/cloudera/impala/analysis/CreateTableAsSelectStmt.java
M fe/src/main/java/com/cloudera/impala/analysis/CreateTableDataSrcStmt.java
M fe/src/main/java/com/cloudera/impala/analysis/CreateTableLikeFileStmt.java
M fe/src/main/java/com/cloudera/impala/analysis/CreateTableStmt.java
A fe/src/main/java/com/cloudera/impala/analysis/DeleteStmt.java
A fe/src/main/java/com/cloudera/impala/analysis/DistributeParam.java
A fe/src/main/java/com/cloudera/impala/analysis/FromClause.java
M fe/src/main/java/com/cloudera/impala/analysis/InsertStmt.java
A fe/src/main/java/com/cloudera/impala/analysis/ModifyStmt.java
M fe/src/main/java/com/cloudera/impala/analysis/SelectStmt.java
M fe/src/main/java/com/cloudera/impala/analysis/SlotDescriptor.java
M fe/src/main/java/com/cloudera/impala/analysis/StmtRewriter.java
M fe/src/main/java/com/cloudera/impala/analysis/ToSqlUtils.java
A fe/src/main/java/com/cloudera/impala/analysis/UpdateStmt.java
M fe/src/main/java/com/cloudera/impala/catalog/CatalogServiceCatalog.java
M fe/src/main/java/com/cloudera/impala/catalog/Column.java
A fe/src/main/java/com/cloudera/impala/catalog/KuduColumn.java
A fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java
M fe/src/main/java/com/cloudera/impala/catalog/Table.java
A fe/src/main/java/com/cloudera/impala/catalog/delegates/DdlDelegate.java
A fe/src/main/java/com/cloudera/impala/catalog/delegates/KuduDdlDelegate.java
A fe/src/main/java/com/cloudera/impala/catalog/delegates/UnsupportedOpDelegate.java
M fe/src/main/java/com/cloudera/impala/planner/DataSink.java
M fe/src/main/java/com/cloudera/impala/planner/DistributedPlanner.java
M fe/src/main/java/com/cloudera/impala/planner/HBaseTableSink.java
M fe/src/main/java/com/cloudera/impala/planner/HdfsTableSink.java
A fe/src/main/java/com/cloudera/impala/planner/KuduScanNode.java
A fe/src/main/java/com/cloudera/impala/planner/KuduTableSink.java
M fe/src/main/java/com/cloudera/impala/planner/Planner.java
M fe/src/main/java/com/cloudera/impala/planner/PlannerContext.java
M fe/src/main/java/com/cloudera/impala/planner/SingleNodePlanner.java
M fe/src/main/java/com/cloudera/impala/planner/TableSink.java
M fe/src/main/java/com/cloudera/impala/service/CatalogOpExecutor.java
M fe/src/main/java/com/cloudera/impala/service/Frontend.java
A fe/src/main/java/com/cloudera/impala/util/KuduUtil.java
M fe/src/main/jflex/sql-scanner.flex
M fe/src/test/java/com/cloudera/impala/analysis/AnalyzeDDLTest.java
A fe/src/test/java/com/cloudera/impala/analysis/AnalyzeModifyStmtsTest.java
M fe/src/test/java/com/cloudera/impala/analysis/ParserTest.java
M fe/src/test/java/com/cloudera/impala/analysis/ToSqlTest.java
A fe/src/test/java/com/cloudera/impala/planner/KuduPlannerTest.java
M fe/src/test/java/com/cloudera/impala/planner/PlannerTestBase.java
M fe/src/test/java/com/cloudera/impala/testutil/CatalogServiceTestCatalog.java
M fe/src/test/java/com/cloudera/impala/testutil/TestUtils.java
A fe/src/test/java/com/cloudera/impala/util/KuduUtilTest.java
A infra/deploy/deploy.py
M testdata/bin/compute-table-stats.sh
M testdata/bin/create-load-data.sh
M testdata/bin/generate-schema-statements.py
M testdata/bin/generate-test-vectors.py
M testdata/cluster/.gitignore
M testdata/cluster/admin
A testdata/cluster/node_templates/cdh5/etc/init.d/kudu-common
A testdata/cluster/node_templates/cdh5/etc/init.d/kudu-master
A testdata/cluster/node_templates/cdh5/etc/init.d/kudu-tserver
A testdata/cluster/node_templates/cdh5/etc/kudu/master.conf.tmpl
A testdata/cluster/node_templates/cdh5/etc/kudu/tserver.conf.tmpl
M testdata/cluster/node_templates/common/etc/init.d/common.tmpl
M testdata/datasets/functional/functional_schema_template.sql
M testdata/datasets/functional/schema_constraints.csv
A testdata/workloads/functional-planner/queries/PlannerTest/kudu-delete.test
A testdata/workloads/functional-planner/queries/PlannerTest/kudu-selectivity.test
A testdata/workloads/functional-planner/queries/PlannerTest/kudu-update.test
A testdata/workloads/functional-planner/queries/PlannerTest/kudu.test
M testdata/workloads/functional-query/functional-query_core.csv
M testdata/workloads/functional-query/functional-query_dimensions.csv
A testdata/workloads/functional-query/queries/QueryTest/create_kudu.test
A testdata/workloads/functional-query/queries/QueryTest/kudu-scan-node.test
A testdata/workloads/functional-query/queries/QueryTest/kudu-show-create.test
A testdata/workloads/functional-query/queries/QueryTest/kudu_alter.test
A testdata/workloads/functional-query/queries/QueryTest/kudu_crud.test
A testdata/workloads/functional-query/queries/QueryTest/kudu_partition_ddl.test
A testdata/workloads/functional-query/queries/QueryTest/kudu_stats.test
M tests/common/skip.py
M tests/common/test_dimensions.py
M tests/metadata/test_ddl.py
M tests/metadata/test_show_create_table.py
M tests/query_test/test_hdfs_fd_caching.py
A tests/query_test/test_kudu.py
M tests/util/test_file_parser.py
M www/common-footer.tmpl
M www/root.tmpl
129 files changed, 9,177 insertions(+), 330 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/36/2536/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2536
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifa7d84ef583fc1d0ea2b656e3c0f41e9775a4659
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: David Ribeiro Alves <da...@cloudera.com>