Hello Lars Volker, Internal Jenkins, Alex Behm,

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

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

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

Change subject: IMPALA-4028: Improve message for improper Sentry config to make 
extra spaces visible.
......................................................................

IMPALA-4028: Improve message for improper Sentry config to make extra spaces 
visible.

When the Sentry config file contained extra spaces it used to be difficult to
determine what went wrong because the offending path was not quoted making the
spaces hard to see. As a result, Impala startup failed.

This patch quotes the bad file path in the error message to make the extra 
spaces
visible.

Change-Id: I3a76b9e4236caa3f2088fba8a9cf0236fced2634
---
M fe/src/main/java/com/cloudera/impala/authorization/SentryConfig.java
M fe/src/test/java/com/cloudera/impala/analysis/AuthorizationTest.java
2 files changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/09/4309/8
-- 
To view, visit http://gerrit.cloudera.org:8080/4309
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3a76b9e4236caa3f2088fba8a9cf0236fced2634
Gerrit-PatchSet: 8
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: davy...@163.com
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: davy...@163.com

Reply via email to