[ 
https://issues.apache.org/jira/browse/SENTRY-361?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14083155#comment-14083155
 ] 

Sravya Tirukkovalur edited comment on SENTRY-361 at 8/1/14 11:13 PM:
---------------------------------------------------------------------

Thanks for the patch Dapeng! Two questions: 
1. The patch doesnt apply( patch: **** malformed patch at line 57:  )
2. Have you tested it out and does it work? I get this:
{noformat}
Using config file 
sentry-dist/target/apache-sentry-1.5.0-incubating-SNAPSHOT-bin/apache-sentry-1.5.0-incubating-SNAPSHOT-bin/conf/sentry-site.xml
SentrySchemaTool:Parsing failed.  Reason: Unrecognized option: -c
{noformat}
Looks related to SENTRY-242, but doesnt seem to work even if I switch the order 
of parameters.


was (Author: sravya):
Thanks for the patch Dapeng! Two questions: 
1. The patch doesnt apply( patch: **** malformed patch at line 57:  )
2. Have you tested it out? I get this:
{noformat}
Using config file 
sentry-dist/target/apache-sentry-1.5.0-incubating-SNAPSHOT-bin/apache-sentry-1.5.0-incubating-SNAPSHOT-bin/conf/sentry-site.xml
SentrySchemaTool:Parsing failed.  Reason: Unrecognized option: -c
{noformat}
Looks related to SENTRY-242, but doesnt seem to work even if I switch the order 
of parameters.

> Sentry server should use sentry-site.xml in conf directory by default
> ---------------------------------------------------------------------
>
>                 Key: SENTRY-361
>                 URL: https://issues.apache.org/jira/browse/SENTRY-361
>             Project: Sentry
>          Issue Type: Improvement
>    Affects Versions: 1.4.0
>            Reporter: Dapeng Sun
>            Assignee: Dapeng Sun
>             Fix For: 1.4.0
>
>         Attachments: SENTRY-361.patch, SENTRY-361_002.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to