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.
---
