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

Posted to notifications@geode.apache.org by "kirklund (GitHub)" <gi...@apache.org> on 2019/04/01 22:35:14 UTC

[GitHub] [geode] kirklund commented on pull request #3386: GEODE-6582: get max message size only once

The downside of making it `static final` is that it becomes untestable. No test can specify or change the value. The alternative would be to move it up a level and just pass `int maxMessageSize` in via the constructor. That keeps this class more testable.

[ Full content available at: https://github.com/apache/geode/pull/3386 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org