> On Dec. 6, 2016, 10:40 p.m., Vadim Spector wrote: > > sentry-service/sentry-service-common/src/gen/thrift/gen-javabean/org/apache/sentry/provider/db/service/thrift/TSentryPrivilege.java, > > line 1031 > > <https://reviews.apache.org/r/54445/diff/2/?file=1578156#file1578156line1031> > > > > 1. What if it's an empty string (i.e. combination of any space > > characterds)? Should probably be treated as missing value? > > > > 2. Should we strip leading and trailing spaces if any?
Yeah, that is an invalid input. Should be addressed. - kalyan kumar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54445/#review158238 ----------------------------------------------------------- On Dec. 6, 2016, 10:32 p.m., kalyan kumar kalvagadda wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54445/ > ----------------------------------------------------------- > > (Updated Dec. 6, 2016, 10:32 p.m.) > > > Review request for sentry, Alexander Kolbasov, Colin Ma, Hao Hao, Vamsee > Yarlagadda, and Vadim Spector. > > > Repository: sentry > > > Description > ------- > > Both the server_name and action feilds are mandatory in previlage request. > I have made changes in the TSentryPrivilege object validation logic in sentry > server which is applied as soon as receiving Thrift message from client. > If the validation is failed exception is raised and appropriate error is > message is sent. > > > Diffs > ----- > > > sentry-service/sentry-service-common/src/gen/thrift/gen-javabean/org/apache/sentry/provider/db/service/thrift/TSentryPrivilege.java > 15b339f > > Diff: https://reviews.apache.org/r/54445/diff/ > > > Testing > ------- > > I have done unit tests using sentry cli thrift client. > > > Thanks, > > kalyan kumar kalvagadda > >
