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

ASF GitHub Bot commented on FLINK-8652:
---------------------------------------

Github user kl0u commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5489#discussion_r168196028
  
    --- Diff: 
flink-queryable-state/flink-queryable-state-client-java/src/main/java/org/apache/flink/queryablestate/client/QueryableStateClient.java
 ---
    @@ -268,7 +268,7 @@ public ExecutionConfig 
setExecutionConfig(ExecutionConfig config) {
                        final String queryableStateName,
                        final int keyHashCode,
                        final byte[] serializedKeyAndNamespace) {
    -           LOG.info("Sending State Request to {}.", remoteAddress);
    +           LOG.debug("Sending State Request to {}.", remoteAddress);
    --- End diff --
    
    We should go for:
    ```
    if (LOG.isDebugEnabled) {
        ....
    }
    ```


> Reduce log level of QueryableStateClient.getKvState() to DEBUG
> --------------------------------------------------------------
>
>                 Key: FLINK-8652
>                 URL: https://issues.apache.org/jira/browse/FLINK-8652
>             Project: Flink
>          Issue Type: Improvement
>          Components: Queryable State
>    Affects Versions: 1.5.0
>            Reporter: Fabian Hueske
>            Assignee: Fabian Hueske
>            Priority: Major
>             Fix For: 1.5.0
>
>
> The {{QueryableStateClient}} logs each state request on {{INFO}} log level. 
> This results in very verbose logging in most applications that use queryable 
> state.
> I propose to reduce the log level to {{DEBUG}}.
> What do you think [~kkl0u], [~aljoscha]?



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to