-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21178/
-----------------------------------------------------------

Review request for sentry and Prasad Mujumdar.


Bugs: Sentry-175
    https://issues.apache.org/jira/browse/Sentry-175


Repository: sentry


Description
-------

* Trivial fix in the script bin/sentry, arguments were not being passed 
correctly
* Do not fail while parsing non option arguments in SentryMain
* allow help (-h) to be passed to sub commands (service/config-tool)


Diffs
-----

  bin/sentry 81b438283f1efbbb83c5699b5cfd6cbe18946677 
  
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java
 bc739adf87ba7d20ada4f4273e57f31b2406d918 
  
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/SentryMain.java 
eb3482b5eb4d88d7ea8bf36b3802fa2ebeebbdf7 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
 bebaf0d234528e90c91d2882dd2a5767d1a61e1d 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
 62584713a3f8d88d4881709357692418c882e43e 

Diff: https://reviews.apache.org/r/21178/diff/


Testing
-------

bin/sentry -h
bin/sentry --command service -h
bin/sentry --command config-tool -h
bin/sentry --command service --conffile /tmp/sentry-site.xml


Thanks,

Sravya Tirukkovalur

Reply via email to