ea2e475185b5863ef6eed347f57286d6a3bfd8a9 | Author: Abe Ratnofsky <aratnofsky@apple.com>
| 2024-02-02 14:56:22-05:00
Address PR feedback: reload-interval to use Optional internally and null in config, rather than using sentinel Duration.ZERO
c7719aed14705b735571ecbfbda23d3b8506eb11 | Author: Abe Ratnofsky <abe@aber.io>
| 2024-01-23 16:09:35-05:00
PR feedback: avoid extra exception wrapping, provide thread naming, improve error messages, etc.
4c7133c72e136d23dbcea795e0041df764568931 | Author: Andy Tolbert <6889771+tolbertam@users.noreply.github.com>
| 2024-01-23 10:21:02-06:00
Replace uses of AttributeKey.newInstance
The java driver uses netty channel attributes to decorate a
connection's channel with the cluster name (returned from
the system.local table) and the map from the OPTIONS response, both
of which are obtained on connection initialization.
There's an issue here that I wouldn't expect to see in practice in
that the AttributeKey's used are created using
AttributeKey.newInstance, which throws an exception if an
AttributeKey of that name is defined anywhere else in evaluated code.
This change attempts to resolve this issue by changing AttributeKey
initialiation in DriverChannel from newInstance to valueOf, which avoids
throwing an exception if an AttributeKey of the same name was previously
instantiated.
patch by Andy Tolbert; reviewed by Bret McGuire, Alexandre Dutra, Abe Ratnofsky for CASSANDRA-19290