You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Riza Suminto (Code Review)" <ge...@cloudera.org> on 2025/06/09 21:36:50 UTC

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Riza Suminto has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22998


Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................

IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3

org.apache.impala.calcite.planner.TpcdsCpuCostPlannerTest fail in S3
build. This test should be skipped in S3, just like the original
org.apache.impala.planner.TpcdsCpuCostPlannerTest. See also IMPALA-8944
on why some tests are skipped in S3.

Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
---
M bin/run-all-tests.sh
1 file changed, 6 insertions(+), 4 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Michael Smith (Code Review)" <ge...@cloudera.org>.
Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22998/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22998/2//COMMIT_MSG@14
PS2, Line 14: This patch also move Calcite FE tests ahead of FE custom cluster tests.
I'd mention this is motivated by making the Calcite JAR available for later tests.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Tue, 10 Jun 2025 15:02:43 +0000
Gerrit-HasComments: Yes

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 2:

Build Successful 

https://jenkins.impala.io/job/gerrit-code-review-checks/19423/ : Initial code review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun to run full precommit tests.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Tue, 10 Jun 2025 02:48:08 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 1:

Build Successful 

https://jenkins.impala.io/job/gerrit-code-review-checks/19421/ : Initial code review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun to run full precommit tests.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Mon, 09 Jun 2025 22:04:19 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Michael Smith (Code Review)" <ge...@cloudera.org>.
Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 3: Verified+1 Code-Review+2


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 3
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Tue, 10 Jun 2025 15:16:30 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Riza Suminto (Code Review)" <ge...@cloudera.org>.
Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22998/1/bin/run-all-tests.sh
File bin/run-all-tests.sh:

http://gerrit.cloudera.org:8080/#/c/22998/1/bin/run-all-tests.sh@325
PS1, Line 325: if ! "${IMPALA_HOME}/bin/mvn-quiet.sh" -fae test install ; then
This is wrong. Even if we don't run planner test, we should still install the jar.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Mon, 09 Jun 2025 23:10:11 +0000
Gerrit-HasComments: Yes

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 2:

Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/11992/ DRY_RUN=true


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Tue, 10 Jun 2025 04:33:13 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Steve Carlin (Code Review)" <ge...@cloudera.org>.
Steve Carlin has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 1: Code-Review+1


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 1
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Mon, 09 Jun 2025 23:02:51 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Riza Suminto (Code Review)" <ge...@cloudera.org>.
Hello Joe McDonnell, Steve Carlin, Impala Public Jenkins, 

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

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

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

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................

IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3

org.apache.impala.calcite.planner.TpcdsCpuCostPlannerTest fail in S3
build. This test should be skipped in S3, just like the original
org.apache.impala.planner.TpcdsCpuCostPlannerTest. See also IMPALA-8944
on why some tests are skipped in S3.

This patch also move Calcite FE tests ahead of FE custom cluster tests.

Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
---
M bin/run-all-tests.sh
1 file changed, 11 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/98/22998/2
-- 
To view, visit http://gerrit.cloudera.org:8080/22998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Impala Public Jenkins (Code Review)" <ge...@cloudera.org>.
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 2: Verified+1


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Tue, 10 Jun 2025 10:35:57 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Riza Suminto (Code Review)" <ge...@cloudera.org>.
Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 4:

Thank you, Michael!


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 4
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Tue, 10 Jun 2025 15:20:00 +0000
Gerrit-HasComments: No

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Riza Suminto (Code Review)" <ge...@cloudera.org>.
Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22998/1/bin/run-all-tests.sh
File bin/run-all-tests.sh:

http://gerrit.cloudera.org:8080/#/c/22998/1/bin/run-all-tests.sh@325
PS1, Line 325: if ! "${IMPALA_HOME}/bin/mvn-quiet.sh" -fae test ${MVN_ARGS}; t
> This is wrong. Even if we don't run planner test, we should still install t
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Tue, 10 Jun 2025 02:24:31 +0000
Gerrit-HasComments: Yes

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Riza Suminto (Code Review)" <ge...@cloudera.org>.
Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................

IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3

org.apache.impala.calcite.planner.TpcdsCpuCostPlannerTest fail in S3
build. This test should be skipped in S3, just like the original
org.apache.impala.planner.TpcdsCpuCostPlannerTest. See also IMPALA-8944
on why some tests are skipped in S3.

This patch also move Calcite FE tests ahead of FE custom cluster tests
so that Calcite JAR is available for later tests.

Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Reviewed-on: http://gerrit.cloudera.org:8080/22998
Reviewed-by: Michael Smith <mi...@cloudera.com>
Tested-by: Michael Smith <mi...@cloudera.com>
---
M bin/run-all-tests.sh
1 file changed, 11 insertions(+), 7 deletions(-)

Approvals:
  Michael Smith: Looks good to me, approved; Verified

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 4
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Riza Suminto (Code Review)" <ge...@cloudera.org>.
Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22998 )

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22998/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22998/2//COMMIT_MSG@14
PS2, Line 14: This patch also move Calcite FE tests ahead of FE custom cluster tests
> I'd mention this is motivated by making the Calcite JAR available for later
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 3
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>
Gerrit-Comment-Date: Tue, 10 Jun 2025 15:06:15 +0000
Gerrit-HasComments: Yes

[Impala-ASF-CR] IMPALA-14135: Skip Calcite planner test if TARGET FILESYSTEM=s3

Posted by "Riza Suminto (Code Review)" <ge...@cloudera.org>.
Hello Joe McDonnell, Steve Carlin, Michael Smith, Impala Public Jenkins, 

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

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

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

Change subject: IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3
......................................................................

IMPALA-14135: Skip Calcite planner test if TARGET_FILESYSTEM=s3

org.apache.impala.calcite.planner.TpcdsCpuCostPlannerTest fail in S3
build. This test should be skipped in S3, just like the original
org.apache.impala.planner.TpcdsCpuCostPlannerTest. See also IMPALA-8944
on why some tests are skipped in S3.

This patch also move Calcite FE tests ahead of FE custom cluster tests
so that Calcite JAR is available for later tests.

Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
---
M bin/run-all-tests.sh
1 file changed, 11 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/98/22998/3
-- 
To view, visit http://gerrit.cloudera.org:8080/22998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8d04915de3eb4c81f7d63f61c1b1d804c9fcd941
Gerrit-Change-Number: 22998
Gerrit-PatchSet: 3
Gerrit-Owner: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <ri...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <sc...@cloudera.com>