Conversation
This reverts commit 71a352f.
olim7t
left a comment
There was a problem hiding this comment.
Implementation looks good to me (pending the additional local DC check requested by Alexandre).
I'm not sure I agree with the change though: we decided to fail fast precisely because we didn't want to miss the fact that some options were going to be ignored. So it sounds consistent that tools would have to add logic to work around that.
The issue is with frameworks that provide some form of support for creating CqlSession instances in a more or less automated way. Take Spring Boot, for instance: the default (And it's not realistic to offer to add support to Astra directly in Spring Boot: Astra being proprietary and commercial, it's likely that such a contribution would meet some resistance to say the least.) |
…ssionBuilder.java Co-authored-by: Alexandre Dutra <adutra@users.noreply.github.com>
…ssionBuilder.java Co-authored-by: Alexandre Dutra <adutra@users.noreply.github.com>
…ssionBuilder.java Co-authored-by: Alexandre Dutra <adutra@users.noreply.github.com>
…e has local-datacenter defined
todo: