Add duplicate check for validateAndTranslateTimeseriesShardKey in createReshardingCoordinatorDoc

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 8.3.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Cluster Scalability
    • Fully Compatible
    • ALL
    • ClusterScalability 2Feb-16Feb
    • 200
    • 1
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      SERVER-111411 refactored the code the validate timeseries shard key early in the flow and removed it from  resharding_util::createReshardingCoordinatorDoc.

      The consequence of this is the following failure scenario in multiversion:

      1. Old coordinator doesn't have early validation → sends invalid timeseries reshard key to config server.
      2. New config server's createReshardingCoordinatorDoc() no longer validates timeseries keys and incorrectly validates the key.

      We should readd this duplicate check until SERVER-111411 is backported till 8.0

            Assignee:
            Abdul Qadeer
            Reporter:
            Abdul Qadeer
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: