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

Alexander Kolbasov commented on SENTRY-1853:
--------------------------------------------

[~davidxdh] Hadoop complains that it can't apply the patch. I tried to apply 
this myself and couldn't. The problem is that the patch has a diff to the 
SentryWebServer.java file which doesn't exist yet - looks like your patch is 
based not on master but on your changes. You need to get a patch against master 
branch.

{code}
$ git apply SENTRY-1853.005.patch 
error: 
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryWebServer.java:
 No such file or directory
{code}

So the problem is with this part:

{code}
--- 
a/sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryWebServer.java
+++ 
b/sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryWebServer.java
@@ -104,6 +104,8 @@ public class SentryWebServer {
     servletContextHandler.getServletContext()
         .setAttribute(ConfServlet.CONF_CONTEXT_ATTRIBUTE, conf);
 
+    servletContextHandler.addServlet(new ServletHolder(LogLevelServlet.class), 
"/admin/logLevel");
+
{code}

Please post a patch against the master branch and once all tests are passing 
I'll commit your change.




> Add the log level access mechanism
> ----------------------------------
>
>                 Key: SENTRY-1853
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1853
>             Project: Sentry
>          Issue Type: New Feature
>            Reporter: Donghui Xu
>            Assignee: Donghui Xu
>         Attachments: SENTRY-1853.001.patch, SENTRY-1853.002.patch, 
> SENTRY-1853.003.patch, SENTRY-1853.004.patch, SENTRY-1853.005.patch
>
>
> Add the log level setting and reading mechanism, through which users can 
> control the log content.
> This mechanism provides the url interface,through which users can dynamically 
> set the log level. Service restores the original log level after restart.
> The format of the interface is as follows:
> 1. Read the log level. URL address is http: //HOST: PORT/admin/logLevel? Log 
> = CLASS_NAME
> 2. Set the log level.  URL address is http: //HOST: PORT/admin/logLevel? Log 
> = CLASS_NAME & level = LOG_LEVEL
> The above parameters include:
> HOST: host name or ip address of sentry server.
> PORT: port of sentry server.
> CLASS_NAME: name of class whose log level is read or set.
> LOG_LEVEL: the log level to be set.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to