| commit | a9ab32e3f14ebef15ee553118bd05f4fc024babe | [log] [tgz] |
|---|---|---|
| author | Murtadha Hubail <mhubail@apache.org> | Fri Oct 18 22:04:05 2019 +0300 |
| committer | Murtadha Hubail <mhubail@apache.org> | Sun Oct 20 11:17:21 2019 +0000 |
| tree | cc996a07a35ea18bd28b7d09dae40553eb72b382 | |
| parent | d7ef925f3c3852bce601903fa7bc6068bf4cc6fa [diff] |
[NO ISSUE][NET] Exclude Warn Logs When No Buffer Factory is Used - user model changes: no - storage format changes: no - interface changes: no Details: - Do not warn about recycled partial frames when no buffer factory is set in FullFrameChannelReadInterface as those are valid cases that use a single frame to perform the complete operation. Change-Id: I80251c529eb15ee22d01c320883418ea80272c2c Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/3824 Reviewed-by: Murtadha Hubail <mhubail@apache.org> Reviewed-by: Till Westmann <tillw@apache.org> Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu> Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
diff --git a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java index 9d7f848..0ce5a87 100644 --- a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java +++ b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/FullFrameChannelReadInterface.java
@@ -47,7 +47,7 @@ credits = 0; emptyBufferAcceptor = buffer -> { final int delta = buffer.remaining(); - if (delta != frameSize) { + if (bufferFactory != null && delta != frameSize) { LOGGER.warn("partial frame being recycled; expected size {}, actual size {}", frameSize, delta); } synchronized (bufferRecycleLock) {