> On March 8, 2016, 12:44 a.m., Li Li wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/ConfServlet.java, > > line 53 > > <https://reviews.apache.org/r/44400/diff/2/?file=1282195#file1282195line53> > > > > For other format, it is handled in line 64: > > will return a HTTP ERROR: 400 with info: Bad format: + format
Cool, thanks! - Hao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44400/#review122429 ----------------------------------------------------------- On March 7, 2016, 4:40 a.m., Li Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44400/ > ----------------------------------------------------------- > > (Updated March 7, 2016, 4:40 a.m.) > > > Review request for sentry, Anne Yu, Hao Hao, Lenni Kuff, and Sravya > Tirukkovalur. > > > Repository: sentry > > > Description > ------- > > Add servlet for dumping configurations > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/ConfServlet.java > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryWebServer.java > 9778c963c023244adfe4c0299ee1b372b1eddbc7 > sentry-provider/sentry-provider-db/src/main/webapp/SentryService.html > f63097307f8359e650d9742d35a4683bec14a894 > > Diff: https://reviews.apache.org/r/44400/diff/ > > > Testing > ------- > > > Thanks, > > Li Li > >
