[JAVA-2941] Use wire version instead of server version to detect wire protocol changes Created: 23/Aug/18 Updated: 28/Oct/23 Resolved: 04/Dec/18 |
|
| Status: | Closed |
| Project: | Java Driver |
| Component/s: | Wire Protocol |
| Affects Version/s: | 3.8.1 |
| Fix Version/s: | 3.10.0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Louis Williams | Assignee: | Jeffrey Yemin |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
When using sessions like they are used in this code, if the server's buildInfo indicates it does not support sessions, in my case 0.0.0 in a development build, the driver will silently fail. As a consequence, the driver accepted my configuration for sessions with causal consistency, but did not provide me causally consistent reads. Using buildInfo to check for features appears to contradict the specification for How to Check Whether a Deployment Supports Sessions. The driver should either report an error in this case or not depend on buildInfo for feature support. |
| Comments |
| Comment by Githook User [ 14/Jan/19 ] |
|
Author: {'username': 'jyemin', 'email': 'jeff.yemin@10gen.com', 'name': 'Jeff Yemin'}Message: Fix a few remaining unit tests These are unit tests hiding in a functional test spec that only runs
|
| Comment by Githook User [ 14/Jan/19 ] |
|
Author: {'username': 'jyemin', 'email': 'jeff.yemin@10gen.com', 'name': 'Jeff Yemin'}Message: Add maxWireVersion property to ConnectionDescription The maxWireVersion becomes a ConnectionDescription property, which is now
|
| Comment by Githook User [ 04/Dec/18 ] |
|
Author: {'name': 'Jeff Yemin', 'email': 'jeff.yemin@10gen.com', 'username': 'jyemin'}Message: Fix a few remaining unit tests These are unit tests hiding in a functional test spec that only runs
|
| Comment by Githook User [ 04/Dec/18 ] |
|
Author: {'name': 'Jeff Yemin', 'email': 'jeff.yemin@10gen.com', 'username': 'jyemin'}Message: Add maxWireVersion property to ConnectionDescription The maxWireVersion becomes a ConnectionDescription property, which is now
|