Uploaded image for project: 'Compass '
  1. Compass
  2. COMPASS-6255

Investigate changes in NODE-4705: Remove Timestamp legacy constructor from low/high order bits

    • Type: Icon: Investigation Investigation
    • Resolution: Done
    • Priority: Icon: Major - P3 Major - P3
    • No version
    • Affects Version/s: None
    • Component/s: None
    • Labels:
    • Not Needed

      Use Case

      We deprecated the 2 argument Timestamp constructor taking low/high order bits. We should remove this constructor support.

      User Impact

      • Migration will require code changes
        • Any instance of new Timestamp(low: number, high: number) will need to be wrapped in Long construction or broken into the i, t style constructor.

      Acceptance Criteria

      Implementation Requirements

      • Remove support for the constructor overload that takes two plain numbers as input
      • Since removing support will likely lead to a runtime error (props not existing etc.) add a helpful error to be thrown if this usage is detected.

      Testing Requirements

      • Assert the constructor fails in the old usage case
      • Update any Timestamp tests to use the i, t style constructor

            Assignee:
            Unassigned Unassigned
            Reporter:
            dbeng-pm-bot PM Bot
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: