[GitHub] lichaojacobs opened a new pull request #326: Fix fe.conf audit_log_moudles,sys_log_roll_mode,audit_log_roll_mode s…

[GitBox <gi...@apache.org>]

lichaojacobs opened a new pull request #326: Fix fe.conf audit_log_moudles,sys_log_roll_mode,audit_log_roll_mode s…
URL: https://github.com/apache/incubator-doris/pull/326
 
 
   I am sorry that i have made a mistake for the last pull request. In fact i just test the FE config.java on my computer locally, and found the problem of fe.conf default values.
   
   1. when i uncomment the `audit_log_modules` variable with value remains the same as source code, i found the value will be parsed like this: `["{"slow_query"",""query}""]
   `
   ![image](https://user-images.githubusercontent.com/7459785/48660083-00d94380-ea97-11e8-8083-998f4396030c.png)
   it makes AuditLog can not locate Logger named `audit.slow_query ,audit.query
   `
   only when i change the audit_log_modules setting like this: `audit_log_modules = slow_query, query`, can the value be parsed in the right way:
   
   ![image](https://user-images.githubusercontent.com/7459785/48660148-d6d45100-ea97-11e8-85c8-cf1288f5345c.png)
   
   
   2. The problem of `audit_log_roll_mode, sys_log_roll_mode` is the same as `audit_log_modules`

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@doris.apache.org
For additional commands, e-mail: dev-help@doris.apache.org