[
https://issues.apache.org/jira/browse/ZOOKEEPER-2517?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Benjamin Jaton updated ZOOKEEPER-2517:
--------------------------------------
Description:
In ClientCnxnSocket.java the parsing of the system property is erroneous:
{code}packetLen = Integer.getInteger(
clientConfig.getProperty(ZKConfig.JUTE_MAXBUFFER),
ZKClientConfig.CLIENT_MAX_PACKET_LENGTH_DEFAULT
);{code}
Javadoc of Integer.getInteger states "The first argument is treated as the name
of a system property", whereas here the value of the property is passed.
Instead I believe the author meant to write something like:
{code}packetLen = Integer.parseInt(
clientConfig.getProperty(
ZKConfig.JUTE_MAXBUFFER,
String.valueOf(ZKClientConfig.CLIENT_MAX_PACKET_LENGTH_DEFAULT)
)
);{code}
was:
In ClientCnxnSocket.java the parsing of the system property is erroneous:
{code}packetLen = Integer.getInteger(
clientConfig.getProperty(ZKConfig.JUTE_MAXBUFFER),
ZKClientConfig.CLIENT_MAX_PACKET_LENGTH_DEFAULT
);{code}
Javadoc of Integer.getInteger states "The first argument is treated as the name
of a system property", whereas here the value of the property is passed.
Instead I believe the author meant to write something like:
{code}packetLen = Integer.getInteger(
clientConfig.getProperty(
ZKConfig.JUTE_MAXBUFFER,
String.valueOf(ZKClientConfig.CLIENT_MAX_PACKET_LENGTH_DEFAULT)
)
);{code}
> jute.maxbuffer is ignored
> -------------------------
>
> Key: ZOOKEEPER-2517
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2517
> Project: ZooKeeper
> Issue Type: Bug
> Affects Versions: 3.5.2
> Reporter: Benjamin Jaton
> Priority: Critical
>
> In ClientCnxnSocket.java the parsing of the system property is erroneous:
> {code}packetLen = Integer.getInteger(
> clientConfig.getProperty(ZKConfig.JUTE_MAXBUFFER),
> ZKClientConfig.CLIENT_MAX_PACKET_LENGTH_DEFAULT
> );{code}
> Javadoc of Integer.getInteger states "The first argument is treated as the
> name of a system property", whereas here the value of the property is passed.
> Instead I believe the author meant to write something like:
> {code}packetLen = Integer.parseInt(
> clientConfig.getProperty(
> ZKConfig.JUTE_MAXBUFFER,
> String.valueOf(ZKClientConfig.CLIENT_MAX_PACKET_LENGTH_DEFAULT)
> )
> );{code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
