[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-29 Thread Internal Jenkins (Code Review) 
Internal Jenkins has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 5: Verified+1

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 5
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Marcel Kornacker 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-29 Thread Internal Jenkins (Code Review) 
Internal Jenkins has submitted this change and it was merged.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


IMPALA-3786: Replace "cloudera" with "apache" (part 1)

As part of the ASF transition, we need to replace references to
Cloudera in Impala with references to Apache. This primarily means
changing Java package names from com.cloudera.impala.* to
org.apache.impala.*

To make this easier to review, this patch only renames files,
eg. fe/src/main/java/com/cloudera -> fe/src/main/java/org/apache

A follow up patch performs the actual code updates.

Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Reviewed-on: http://gerrit.cloudera.org:8080/3936
Reviewed-by: Thomas Tauber-Marshall 
Reviewed-by: Jim Apple 
Tested-by: Internal Jenkins
---
R 
ext-data-source/api/src/main/java/org/apache/impala/extdatasource/util/SerializationUtils.java
R 
ext-data-source/api/src/main/java/org/apache/impala/extdatasource/v1/ExternalDataSource.java
R 
ext-data-source/sample/src/main/java/org/apache/impala/extdatasource/sample/EchoDataSource.java
R 
ext-data-source/test/src/main/java/org/apache/impala/extdatasource/AllTypesDataSource.java
R fe/src/main/java/org/apache/impala/analysis/AggregateInfo.java
R fe/src/main/java/org/apache/impala/analysis/AggregateInfoBase.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableAddPartitionStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableAddReplaceColsStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableChangeColStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableDropColStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableDropPartitionStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableOrViewRenameStmt.java
R 
fe/src/main/java/org/apache/impala/analysis/AlterTableRecoverPartitionsStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetCachedStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetColumnStats.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetFileFormatStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetLocationStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetTblProperties.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterViewStmt.java
R fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticExpr.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticInfo.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticWindow.java
R fe/src/main/java/org/apache/impala/analysis/Analyzer.java
R fe/src/main/java/org/apache/impala/analysis/ArithmeticExpr.java
R fe/src/main/java/org/apache/impala/analysis/AuthorizationStmt.java
R fe/src/main/java/org/apache/impala/analysis/BaseTableRef.java
R fe/src/main/java/org/apache/impala/analysis/BetweenPredicate.java
R fe/src/main/java/org/apache/impala/analysis/BinaryPredicate.java
R fe/src/main/java/org/apache/impala/analysis/BoolLiteral.java
R fe/src/main/java/org/apache/impala/analysis/CaseExpr.java
R fe/src/main/java/org/apache/impala/analysis/CaseWhenClause.java
R fe/src/main/java/org/apache/impala/analysis/CastExpr.java
R fe/src/main/java/org/apache/impala/analysis/CollectionStructType.java
R fe/src/main/java/org/apache/impala/analysis/CollectionTableRef.java
R fe/src/main/java/org/apache/impala/analysis/ColumnDef.java
R fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java
R fe/src/main/java/org/apache/impala/analysis/CompoundPredicate.java
R fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDataSrcStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDbStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDropRoleStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java
R fe/src/main/java/org/apache/impala/analysis/CreateOrAlterViewStmtBase.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableAsSelectStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableDataSrcStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableLikeFileStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableLikeStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateUdaStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateUdfStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateViewStmt.java
R fe/src/main/java/org/apache/impala/analysis/DeleteStmt.java
R fe/src/main/java/org/apache/impala/analysis/DescribeDbStmt.java
R fe/src/main/java/org/apache/impala/analysis/DescribeTableStmt.java
R

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-29 Thread Jim Apple (Code Review) 
Jim Apple has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 5: Code-Review+2

Carry ALex's +2

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 5
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Marcel Kornacker 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-29 Thread Thomas Tauber-Marshall (Code Review) 
Thomas Tauber-Marshall has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 5: Code-Review+1

Rebased, carrying forward +1s.

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 5
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Marcel Kornacker 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-29 Thread Thomas Tauber-Marshall (Code Review) 
Hello Jim Apple, Alex Behm, Dan Hecht,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#5).

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..

IMPALA-3786: Replace "cloudera" with "apache" (part 1)

As part of the ASF transition, we need to replace references to
Cloudera in Impala with references to Apache. This primarily means
changing Java package names from com.cloudera.impala.* to
org.apache.impala.*

To make this easier to review, this patch only renames files,
eg. fe/src/main/java/com/cloudera -> fe/src/main/java/org/apache

A follow up patch performs the actual code updates.

Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
---
R 
ext-data-source/api/src/main/java/org/apache/impala/extdatasource/util/SerializationUtils.java
R 
ext-data-source/api/src/main/java/org/apache/impala/extdatasource/v1/ExternalDataSource.java
R 
ext-data-source/sample/src/main/java/org/apache/impala/extdatasource/sample/EchoDataSource.java
R 
ext-data-source/test/src/main/java/org/apache/impala/extdatasource/AllTypesDataSource.java
R fe/src/main/java/org/apache/impala/analysis/AggregateInfo.java
R fe/src/main/java/org/apache/impala/analysis/AggregateInfoBase.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableAddPartitionStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableAddReplaceColsStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableChangeColStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableDropColStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableDropPartitionStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableOrViewRenameStmt.java
R 
fe/src/main/java/org/apache/impala/analysis/AlterTableRecoverPartitionsStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetCachedStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetColumnStats.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetFileFormatStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetLocationStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetTblProperties.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterViewStmt.java
R fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticExpr.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticInfo.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticWindow.java
R fe/src/main/java/org/apache/impala/analysis/Analyzer.java
R fe/src/main/java/org/apache/impala/analysis/ArithmeticExpr.java
R fe/src/main/java/org/apache/impala/analysis/AuthorizationStmt.java
R fe/src/main/java/org/apache/impala/analysis/BaseTableRef.java
R fe/src/main/java/org/apache/impala/analysis/BetweenPredicate.java
R fe/src/main/java/org/apache/impala/analysis/BinaryPredicate.java
R fe/src/main/java/org/apache/impala/analysis/BoolLiteral.java
R fe/src/main/java/org/apache/impala/analysis/CaseExpr.java
R fe/src/main/java/org/apache/impala/analysis/CaseWhenClause.java
R fe/src/main/java/org/apache/impala/analysis/CastExpr.java
R fe/src/main/java/org/apache/impala/analysis/CollectionStructType.java
R fe/src/main/java/org/apache/impala/analysis/CollectionTableRef.java
R fe/src/main/java/org/apache/impala/analysis/ColumnDef.java
R fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java
R fe/src/main/java/org/apache/impala/analysis/CompoundPredicate.java
R fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDataSrcStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDbStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDropRoleStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java
R fe/src/main/java/org/apache/impala/analysis/CreateOrAlterViewStmtBase.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableAsSelectStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableDataSrcStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableLikeFileStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableLikeStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateUdaStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateUdfStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateViewStmt.java
R fe/src/main/java/org/apache/impala/analysis/DeleteStmt.java
R fe/src/main/java/org/apache/impala/analysis/DescribeDbStmt.java
R fe/src/main/java/org/apache/impala/analysis/DescribeTableStmt.java
R fe/src/main/java/org/apache/impala/analysis/DescriptorTable.java
R

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-27 Thread Alex Behm (Code Review) 
Alex Behm has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 4: Code-Review+2

Trivial enough that I feel giving a +2 on the catalog changes also.

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Marcel Kornacker 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-27 Thread Dan Hecht (Code Review) 
Dan Hecht has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 4:

Same thing with this one.

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Henry Robinson 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Marcel Kornacker 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-26 Thread Thomas Tauber-Marshall (Code Review) 
Thomas Tauber-Marshall has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 4:

Jim - Just waiting on someone to +2 it. You had added Alex to the review last 
week, but he's been out since then.

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-26 Thread Jim Apple (Code Review) 
Jim Apple has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 4: Code-Review+1

Carry Dan's +1.

Thomas, is this waiting on anything?

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-20 Thread Thomas Tauber-Marshall (Code Review) 
Hello Jim Apple, Dan Hecht,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#4).

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..

IMPALA-3786: Replace "cloudera" with "apache" (part 1)

As part of the ASF transition, we need to replace references to
Cloudera in Impala with references to Apache. This primarily means
changing Java package names from com.cloudera.impala.* to
org.apache.impala.*

To make this easier to review, this patch only renames files,
eg. fe/src/main/java/com/cloudera -> fe/src/main/java/org/apache

A follow up patch performs the actual code updates.

Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
---
R 
ext-data-source/api/src/main/java/org/apache/impala/extdatasource/util/SerializationUtils.java
R 
ext-data-source/api/src/main/java/org/apache/impala/extdatasource/v1/ExternalDataSource.java
R 
ext-data-source/sample/src/main/java/org/apache/impala/extdatasource/sample/EchoDataSource.java
R 
ext-data-source/test/src/main/java/org/apache/impala/extdatasource/AllTypesDataSource.java
R fe/src/main/java/org/apache/impala/analysis/AggregateInfo.java
R fe/src/main/java/org/apache/impala/analysis/AggregateInfoBase.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableAddPartitionStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableAddReplaceColsStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableChangeColStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableDropColStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableDropPartitionStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableOrViewRenameStmt.java
R 
fe/src/main/java/org/apache/impala/analysis/AlterTableRecoverPartitionsStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetCachedStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetColumnStats.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetFileFormatStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetLocationStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableSetTblProperties.java
R fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java
R fe/src/main/java/org/apache/impala/analysis/AlterViewStmt.java
R fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticExpr.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticInfo.java
R fe/src/main/java/org/apache/impala/analysis/AnalyticWindow.java
R fe/src/main/java/org/apache/impala/analysis/Analyzer.java
R fe/src/main/java/org/apache/impala/analysis/ArithmeticExpr.java
R fe/src/main/java/org/apache/impala/analysis/AuthorizationStmt.java
R fe/src/main/java/org/apache/impala/analysis/BaseTableRef.java
R fe/src/main/java/org/apache/impala/analysis/BetweenPredicate.java
R fe/src/main/java/org/apache/impala/analysis/BinaryPredicate.java
R fe/src/main/java/org/apache/impala/analysis/BoolLiteral.java
R fe/src/main/java/org/apache/impala/analysis/CaseExpr.java
R fe/src/main/java/org/apache/impala/analysis/CaseWhenClause.java
R fe/src/main/java/org/apache/impala/analysis/CastExpr.java
R fe/src/main/java/org/apache/impala/analysis/CollectionStructType.java
R fe/src/main/java/org/apache/impala/analysis/CollectionTableRef.java
R fe/src/main/java/org/apache/impala/analysis/ColumnDef.java
R fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java
R fe/src/main/java/org/apache/impala/analysis/CompoundPredicate.java
R fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDataSrcStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDbStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateDropRoleStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java
R fe/src/main/java/org/apache/impala/analysis/CreateOrAlterViewStmtBase.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableAsSelectStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableDataSrcStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableLikeFileStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableLikeStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateTableStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateUdaStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateUdfStmt.java
R fe/src/main/java/org/apache/impala/analysis/CreateViewStmt.java
R fe/src/main/java/org/apache/impala/analysis/DeleteStmt.java
R fe/src/main/java/org/apache/impala/analysis/DescribeDbStmt.java
R fe/src/main/java/org/apache/impala/analysis/DescribeTableStmt.java
R fe/src/main/java/org/apache/impala/analysis/DescriptorTable.java
R

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-20 Thread Jim Apple (Code Review) 
Jim Apple has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 3:

> I have finally gotten the BVTs to successfully run.
 > 
 > Packaging run:
 > http://golden.jenkins.cloudera.com/job/CDH5-Packaging-On-Demand/3043/
 > 
 > That failed due to a timeout while waiting for the BVT job, but the
 > BVT job itself ran and succeeded:
 > http://qe.jenkins.cloudera.com/job/docker-build_cdh_cluster/957/
 > 
 > and the packaging job doesn't have any significant steps after the
 > BVT job, so if it hadn't timed out everything would have succeeded.

Dan, what do you think?

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-09-16 Thread Thomas Tauber-Marshall (Code Review) 
Thomas Tauber-Marshall has posted comments on this change.

Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1)
..


Patch Set 3:

I have finally gotten the BVTs to successfully run.

Packaging run:
http://golden.jenkins.cloudera.com/job/CDH5-Packaging-On-Demand/3043/

That failed due to a timeout while waiting for the BVT job, but the BVT job 
itself ran and succeeded:
http://qe.jenkins.cloudera.com/job/docker-build_cdh_cluster/957/

and the packaging job doesn't have any significant steps after the BVT job, so 
if it hadn't timed out everything would have succeeded.

-- 
To view, visit http://gerrit.cloudera.org:8080/3936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3767dd1ee86df767075fdf1b371eb6b0b06668db
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall 
Gerrit-Reviewer: Dan Hecht 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No