You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by zentol <gi...@git.apache.org> on 2018/03/08 19:34:16 UTC

[GitHub] flink pull request #5594: [FLINK-8800][REST] Reduce logging of all requests ...

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

    https://github.com/apache/flink/pull/5594#discussion_r173266836
  
    --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/rest/AbstractHandler.java ---
    @@ -84,8 +84,8 @@ protected AbstractHandler(
     
     	@Override
     	protected void respondAsLeader(ChannelHandlerContext ctx, Routed routed, T gateway) throws Exception {
    -		if (log.isDebugEnabled()) {
    -			log.debug("Received request " + routed.request().getUri() + '.');
    +		if (log.isTraceEnabled()) {
    --- End diff --
    
    Yes we could simplify this, however we would be relying on implementation details of 2 libraries.


---