{"action":"created","issue":{"url":"https://api.github.com/repos/influxdata/influxdb-java/issues/450","repository_url":"https://api.github.com/repos/influxdata/influxdb-java","labels_url":"https://api.github.com/repos/influxdata/influxdb-java/issues/450/labels{/name}","comments_url":"https://api.github.com/repos/influxdata/influxdb-java/issues/450/comments","events_url":"https://api.github.com/repos/influxdata/influxdb-java/issues/450/events","html_url":"https://github.com/influxdata/influxdb-java/pull/450","id":325957300,"number":450,"title":"issues #413 : Debug mode which allows HTTP requests being sent to the…","user":{"login":"lxhoan","id":1539343,"avatar_url":"https://avatars2.githubusercontent.com/u/1539343?v=4","gravatar_id":"","url":"https://api.github.com/users/lxhoan","html_url":"https://github.com/lxhoan","followers_url":"https://api.github.com/users/lxhoan/followers","following_url":"https://api.github.com/users/lxhoan/following{/other_user}","gists_url":"https://api.github.com/users/lxhoan/gists{/gist_id}","starred_url":"https://api.github.com/users/lxhoan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lxhoan/subscriptions","organizations_url":"https://api.github.com/users/lxhoan/orgs","repos_url":"https://api.github.com/users/lxhoan/repos","events_url":"https://api.github.com/users/lxhoan/events{/privacy}","received_events_url":"https://api.github.com/users/lxhoan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-05-24T04:10:43Z","updated_at":"2018-05-30T16:11:51Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/influxdata/influxdb-java/pulls/450","html_url":"https://github.com/influxdata/influxdb-java/pull/450","diff_url":"https://github.com/influxdata/influxdb-java/pull/450.diff","patch_url":"https://github.com/influxdata/influxdb-java/pull/450.patch"},"body":"… database to be logged"},"comment":{"url":"https://api.github.com/repos/influxdata/influxdb-java/issues/comments/393221412","html_url":"https://github.com/influxdata/influxdb-java/pull/450#issuecomment-393221412","issue_url":"https://api.github.com/repos/influxdata/influxdb-java/issues/450","id":393221412,"user":{"login":"lxhoan","id":1539343,"avatar_url":"https://avatars2.githubusercontent.com/u/1539343?v=4","gravatar_id":"","url":"https://api.github.com/users/lxhoan","html_url":"https://github.com/lxhoan","followers_url":"https://api.github.com/users/lxhoan/followers","following_url":"https://api.github.com/users/lxhoan/following{/other_user}","gists_url":"https://api.github.com/users/lxhoan/gists{/gist_id}","starred_url":"https://api.github.com/users/lxhoan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lxhoan/subscriptions","organizations_url":"https://api.github.com/users/lxhoan/orgs","repos_url":"https://api.github.com/users/lxhoan/repos","events_url":"https://api.github.com/users/lxhoan/events{/privacy}","received_events_url":"https://api.github.com/users/lxhoan/received_events","type":"User","site_admin":false},"created_at":"2018-05-30T16:11:51Z","updated_at":"2018-05-30T16:11:51Z","author_association":"CONTRIBUTOR","body":"Just want to quote here this (from a discussion between @fmachado and me)\r\n\r\n>As far as I know JMX is a technology for managing and monitoring rather than debugging the application. For debugging you would like to use a more flexible but simple way, and use of a System property here would be better. Two use cases:\r\n1/ You can pass the log level to JVM at booting up\r\n2/ During your debugging session, you can manage to set the log level of the client which is going to created somewhere else (I do not reject you can use JMX but it will never have such a flexibility like evaluating a java statement to change that System property value)\r\n\r\nSo unless we change the original issue to something like __Operation management and monitoring mechanism for influxdb-java ... __ , I would keep on with System property. There are thousands of real practices we can find easily on the Internet for using of System property to configure certain behaviour(s) of an application. Hence, I think I do not need to list some of them here.\r\n\r\nHowever I agree with that I will remove the Use Case /2. So in case if this system property exists, any change to it's value  will not take effect and we can set the logLevel in a static context which is executed only one time. I hope this change is aligned with the review\r\n\r\n>This method is being called every time an object is created. Can we call it just once, statically?\r\n\r\n"}}
{"actor":{"display_login":"lxhoan"}}