Lars Volker has posted comments on this change.

Change subject: IMPALA-4028: Trim sentry config file path spaces while impala 
start.
......................................................................


Patch Set 2:

(1 comment)

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

PS2, Line 9: contains spaces
Your change actually only seems to address leading and trailing white space. Do 
you know what happens if the file path contains a space somewhere in the middle?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3a76b9e4236caa3f2088fba8a9cf0236fced2634
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: [email protected]
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-HasComments: Yes

Reply via email to