[Impala-ASF-CR] IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog
Tianyi Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9228 Change subject: IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog .. IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog After IMPALA-5990, catalog keeps printing "NativeAddPendingTopicItem failed" into the log because of the wrongly implemented error handling. This patch fixes this problem. Change-Id: Ieb5847a698768c704346371288544a54055bdf1d --- M be/src/service/fe-support.cc 1 file changed, 8 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/28/9228/1 -- To view, visit http://gerrit.cloudera.org:8080/9228 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ieb5847a698768c704346371288544a54055bdf1d Gerrit-Change-Number: 9228 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi Wang
[Impala-ASF-CR] IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog
Dimitris Tsirogiannis has posted comments on this change. ( http://gerrit.cloudera.org:8080/9228 ) Change subject: IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9228 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ieb5847a698768c704346371288544a54055bdf1d Gerrit-Change-Number: 9228 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi Wang Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Comment-Date: Tue, 06 Feb 2018 22:54:02 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9228 ) Change subject: IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1896/ -- To view, visit http://gerrit.cloudera.org:8080/9228 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ieb5847a698768c704346371288544a54055bdf1d Gerrit-Change-Number: 9228 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi Wang Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins Gerrit-Comment-Date: Wed, 07 Feb 2018 00:22:55 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9228 ) Change subject: IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog .. Patch Set 1: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1896/ -- To view, visit http://gerrit.cloudera.org:8080/9228 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ieb5847a698768c704346371288544a54055bdf1d Gerrit-Change-Number: 9228 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi Wang Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins Gerrit-Comment-Date: Wed, 07 Feb 2018 04:04:29 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9228 ) Change subject: IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1900/ -- To view, visit http://gerrit.cloudera.org:8080/9228 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ieb5847a698768c704346371288544a54055bdf1d Gerrit-Change-Number: 9228 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi Wang Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins Gerrit-Comment-Date: Wed, 07 Feb 2018 18:06:16 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9228 ) Change subject: IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/9228 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ieb5847a698768c704346371288544a54055bdf1d Gerrit-Change-Number: 9228 Gerrit-PatchSet: 1 Gerrit-Owner: Tianyi Wang Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins Gerrit-Comment-Date: Wed, 07 Feb 2018 21:59:43 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9228 ) Change subject: IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog .. IMPALA-6478: Remove garbage NativeAddPendingTopicItem log from catalog After IMPALA-5990, catalog keeps printing "NativeAddPendingTopicItem failed" into the log because of the wrongly implemented error handling. This patch fixes this problem. Change-Id: Ieb5847a698768c704346371288544a54055bdf1d Reviewed-on: http://gerrit.cloudera.org:8080/9228 Reviewed-by: Dimitris Tsirogiannis Tested-by: Impala Public Jenkins --- M be/src/service/fe-support.cc 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Dimitris Tsirogiannis: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9228 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ieb5847a698768c704346371288544a54055bdf1d Gerrit-Change-Number: 9228 Gerrit-PatchSet: 2 Gerrit-Owner: Tianyi Wang Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins