[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 4: Verified+1


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 
Gerrit-Comment-Date: Thu, 30 May 2019 22:34:16 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..

IMPALA-8333: Remove Impala Shell warnings part 2

Set IMPALA_TOOLCHAIN_BUILD_ID=40-193a30b3af to pickup "Patch Thrift to
0.9.3-p6 to eliminate ssl warnings"

Set IMPALA_THRIFT_VERSION=0.9.3-p6 to pick up the new thrift build,
which removes an unnecessary and confusing warning.

TESTING

Change the tests in test_client_ssl.py which were looking for specific
deprecation warnings to instead search for any Deprecation Warning.
Ran all end-to-end tests with new toolchain.
Built Impala on all supported platforms.

Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Reviewed-on: http://gerrit.cloudera.org:8080/13404
Reviewed-by: Impala Public Jenkins 
Tested-by: Impala Public Jenkins 
---
M bin/impala-config.sh
M tests/custom_cluster/test_client_ssl.py
2 files changed, 6 insertions(+), 11 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 5
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 4: Code-Review+2


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 
Gerrit-Comment-Date: Thu, 30 May 2019 17:05:26 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-30 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 4:

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


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 
Gerrit-Comment-Date: Thu, 30 May 2019 17:05:27 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-29 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 3: Verified-1

Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/4340/


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 
Gerrit-Comment-Date: Wed, 29 May 2019 22:50:21 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-29 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 3:

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


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 
Gerrit-Comment-Date: Wed, 29 May 2019 17:24:45 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-29 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 3: Code-Review+2


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 
Gerrit-Comment-Date: Wed, 29 May 2019 17:24:44 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-28 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 
Gerrit-Comment-Date: Tue, 28 May 2019 18:28:32 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-28 Thread Andrew Sherman (Code Review)
Andrew Sherman has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 2:

No-one picked this up so I am picking on you (Thomas) as you reviewed the 
toolchain change


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Thomas Marshall 
Gerrit-Comment-Date: Tue, 28 May 2019 16:20:13 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13404 )

Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..


Patch Set 2:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/3359/ : 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/13404
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Sherman 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Comment-Date: Thu, 23 May 2019 17:39:06 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-8333: Remove Impala Shell warnings part 2

2019-05-23 Thread Andrew Sherman (Code Review)
Andrew Sherman has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/13404


Change subject: IMPALA-8333: Remove Impala Shell warnings part 2
..

IMPALA-8333: Remove Impala Shell warnings part 2

Set IMPALA_TOOLCHAIN_BUILD_ID=40-193a30b3af to pickup "Patch Thrift to
0.9.3-p6 to eliminate ssl warnings"

Set IMPALA_THRIFT_VERSION=0.9.3-p6 to pick up the new thrift build,
which removes an unnecessary and confusing warning.

TESTING

Change the tests in test_client_ssl.py which were looking for specific
deprecation warnings to instead search for any Deprecation Warning.
Ran all end-to-end tests with new toolchain.
Built Impala on all supported platforms.

Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
---
M bin/impala-config.sh
M tests/custom_cluster/test_client_ssl.py
2 files changed, 6 insertions(+), 11 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8ae7e068894da5981fc083e690051da268bfde4d
Gerrit-Change-Number: 13404
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Sherman