You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Anonymous Coward (Code Review)" <ge...@cloudera.org> on 2016/09/07 13:17:27 UTC
[Impala-ASF-CR] IMPALA-4028: Trim sentry config file path spaces while impala start.
davyxdh@163.com has uploaded a new change for review.
http://gerrit.cloudera.org:8080/4323
Change subject: IMPALA-4028: Trim sentry config file path spaces while impala start.
......................................................................
IMPALA-4028: Trim sentry config file path spaces while impala start.
When sentry config file path contains spaces, impala start up failed.
Trimming sentry-site.xml file path spaces will solve this problem.
Change-Id: I979e509057849345fab2ffb87a83aef2605efaf3
---
M fe/src/main/java/com/cloudera/impala/authorization/SentryConfig.java
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/23/4323/1
--
To view, visit http://gerrit.cloudera.org:8080/4323
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I979e509057849345fab2ffb87a83aef2605efaf3
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: davyxdh@163.com
[Impala-ASF-CR] IMPALA-4028: Trim sentry config file path spaces while impala start.
Posted by "Lars Volker (Code Review)" <ge...@cloudera.org>.
Lars Volker has posted comments on this change.
Change subject: IMPALA-4028: Trim sentry config file path spaces while impala start.
......................................................................
Patch Set 1:
Why did you push this as a new change instead of a patch set to this one? https://gerrit.cloudera.org/#/c/4309/
--
To view, visit http://gerrit.cloudera.org:8080/4323
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I979e509057849345fab2ffb87a83aef2605efaf3
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: davyxdh@163.com
Gerrit-Reviewer: Lars Volker <lv...@cloudera.com>
Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4028: Trim sentry config file path spaces while impala start.
Posted by "Lars Volker (Code Review)" <ge...@cloudera.org>.
Lars Volker has posted comments on this change.
Change subject: IMPALA-4028: Trim sentry config file path spaces while impala start.
......................................................................
Patch Set 1:
Perfect, I see you figured it out. In general, you just need to preserve the Change-Id: between subsequent pushes and Gerrit will automatically add it as a new patch set. Thanks for fixing this.
--
To view, visit http://gerrit.cloudera.org:8080/4323
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I979e509057849345fab2ffb87a83aef2605efaf3
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: davyxdh@163.com
Gerrit-Reviewer: Lars Volker <lv...@cloudera.com>
Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4028: Trim sentry config file path spaces while impala start.
Posted by "Anonymous Coward (Code Review)" <ge...@cloudera.org>.
davyxdh@163.com has abandoned this change.
Change subject: IMPALA-4028: Trim sentry config file path spaces while impala start.
......................................................................
Abandoned
--
To view, visit http://gerrit.cloudera.org:8080/4323
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: abandon
Gerrit-Change-Id: I979e509057849345fab2ffb87a83aef2605efaf3
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: davyxdh@163.com
Gerrit-Reviewer: Lars Volker <lv...@cloudera.com>