----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36535/ -----------------------------------------------------------
Review request for sentry and Vamsee Yarlagadda. Repository: sentry Description ------- I went over the latest CoreAdminHandler code in Solr 4.10 and foudn some minor discrepancies in what we are doing. Biggest one is that REQUESTATUS is classified as an UPDATE request instead of a QUERY request. Diffs ----- sentry-solr/solr-sentry-handlers/src/main/java/org/apache/solr/handler/admin/SecureCoreAdminHandler.java c1bde310e46e9dac7fa7814080e9b36c0d1dcd28 sentry-solr/solr-sentry-handlers/src/test/java/org/apache/solr/handler/admin/SecureCoreAdminHandlerTest.java 1857feb6622f72463f5c465698935b540116fa36 Diff: https://reviews.apache.org/r/36535/diff/ Testing ------- Ran the solr-sentry unit tests. Thanks, Gregory Chanan
