JAVA-1042: Include DSE version and workload in Host data. #580
Conversation
There was a problem hiding this comment.
I've noticed that this can happen even when using DSE as the information isn't populated immediately as the host becomes available on 9042, but eventually the data is populated. This could mean the driver may also not reliably have this data even if using DSE, but doesn't seem likely (except maybe in bootstrap cases). Do we have a way to force refresh of this data? That might be nice to have (this would be a separate request).
There was a problem hiding this comment.
It looks like I was wrong about this, the problem was the control host not being set, after that was fixed everything seems to be fine.
There was a problem hiding this comment.
Seems like i was somewhat right in that it isn't populated immediately on node start, so having a nice API for forcing a refresh on this might be nice (as a separate pr later).
There was a problem hiding this comment.
So after doing a little more testing, it seems that newer DSE versions handles this much better. On 4.8.3 I saw that a Solr workload never sets the dse_version column, where it works just fine.
|
Added tests for this and JAVA-1035. Also made a change to refreshNodeListAndTokenMap to handle listen_address in peers table (which C* doesn't currently provide). |
|
👍 |
This commit contains contributions by Andrew Tolbert (@tolbertam), in particular tests for both JAVA-1042 and JAVA-1035.
JAVA-1042: Include DSE version and workload in Host data.
This PR applies the patch by @djatnieks with additional minor changes to the javadocs of
Hostclass.