eolivelli commented on a change in pull request #1410: Issue #1409: Added
server side backpressure (@bug W-3651831@)
URL: https://github.com/apache/bookkeeper/pull/1410#discussion_r188853946
##########
File path:
bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.java
##########
@@ -128,7 +128,6 @@
protected static final String PCBC_TIMEOUT_TIMER_NUM_TICKS =
"pcbcTimeoutTimerNumTicks";
protected static final String TIMEOUT_TIMER_TICK_DURATION_MS =
"timeoutTimerTickDurationMs";
protected static final String TIMEOUT_TIMER_NUM_TICKS =
"timeoutTimerNumTicks";
- protected static final String WAIT_TIMEOUT_ON_BACKPRESSURE =
"waitTimeoutOnBackpressureMs";
Review comment:
I this it would be better to have two separate properties, one for server
and another for client.
Having two properties with the same name will be confiusing and/or we have
to write docs to explain this fact
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
