[JAVA-4961] Deprecate sharded-replicaset topology type and clean up redundant runOnRequirements Created: 10/May/23  Updated: 25/May/23  Resolved: 25/May/23

Status: Closed
Project: Java Driver
Component/s: Change Streams, CSOT, Retryability, Sessions, Stable API, Transactions, Unified Test Runner
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Minor - P4
Reporter: PM Bot Assignee: Unassigned
Resolution: Won't Fix Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Issue split
split from DRIVERS-2609 Deprecate sharded-replicaset topology... Implementing
Upstream Changes Summary:

DRIVERS-2609:
Sync spec tests to mongodb/specifications@474ddfc for the following directories:

  • change-streams/tests/unified
  • client-side-operations-timeout/tests
  • retryable-writes/tests/unified
  • sessions/tests
  • transactions/tests/unified
  • unified-test-format/tests/valid-pass
  • versioned-api/tests

No test runner changes should be required.

Documentation Changes Summary:

1. What would you like to communicate to the user about this feature?
2. Would you like the user to see examples of the syntax and/or executable code and its output?
3. Which versions of the driver/connector does this apply to?


 Description   

This ticket was split from DRIVERS-2609, please see that ticket for a detailed description.



 Comments   
Comment by Jeffrey Yemin [ 22/May/23 ]

On further reflection, I don't see any reason to do this as there is no technical debt incurred by keeping it as is.  It's just one conditional statement that could be simplified from

(typeString.equals("sharded") || typeString.equals("sharded-replicaset")) && isSharded()

to

typeString.equals("sharded") && isSharded()

Generated at Thu Feb 08 09:03:23 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.