Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-3126

Improve BigInt Serialization

XMLWordPrintableJSON

    • Type: Icon: Epic Epic
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 5.1.0
    • Affects Version/s: None
    • Component/s: None
    • Not Needed
    • Hide
      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?
      Show
      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?
    • Done
    • Improve BigInt Serialization
    • Hide

      Engineer: Warren James
      Summary: This epic captures several tickets that have cropped up from users indicating that we need to improve how we serialize BigInts.

      2023-03-02: Project complete!

      Engineer: Warren James
      2023-02-17: Target date pushed out to Feb 24

      • This week, we fixed the EJSON.parse bug (NODE-5056) and unblocked execution on getting BigInt support into the driver (NODE-4877).
      • Continuing to work on resolving primary review comments for that ticket.
      • Next week, looking to wrap NODE-4877 and close out the epic.

      Engineer: Warren James
      2023-02-02: Target date pushed out to Feb 10

      • Status update:
        • The ESLint plugin, as well as testing the runtime without BigInt was put into review, that cycle is in progress, we're adding a new EVG task so all BSON tests will run without bigint
        • Working to get BigInt passing on the driver's CI, expect this to happen mid next week. 
      • Rationale for delays:
        • BigInt in the driver PR was put down somewhat to focus on other pressing bugs but the CI patches ran revealed the areas we need to put logic to get this working
      • Risks:
        • No risks.

      2023-01-20:

      • Status update:
        • Working on support for BSON useBigInt64 flag to Node Driver 
      • Rationale for delays:
        • No delays.
      • Risks:
        • No risks.

      2023-01-06:

      • Status update:
        • Working on EJSON parse/stringify support for BigInt to/from $numberLong
      • Rationale for delays:
        • No delays.
      • Risks:
        • No risks.

      2022-12-23:

      • Status update:
      1. Support BigInt64 deserialization in js-bson in review.
      • Rationale for delays:
      1. No delays.
      • Risks:
      1. No risks.

      2022-12-09:

      • Status update:
        • Started project, working on breakdown of project tickets
      • Rationale for delays:
        • No delays.
      • Risks:
        • No risks.
      Show
      Engineer: Warren James Summary: This epic captures several tickets that have cropped up from users indicating that we need to improve how we serialize BigInts. 2023-03-02: Project complete! Engineer: Warren James 2023-02-17: Target date pushed out to Feb 24 This week, we fixed the EJSON.parse bug ( NODE-5056 ) and unblocked execution on getting BigInt support into the driver ( NODE-4877 ). Continuing to work on resolving primary review comments for that ticket. Next week, looking to wrap NODE-4877 and close out the epic. Engineer: Warren James 2023-02-02: Target date pushed out to Feb 10 Status update: The ESLint plugin, as well as testing the runtime without BigInt was put into review, that cycle is in progress, we're adding a new EVG task so all BSON tests will run without bigint Working to get BigInt passing on the driver's CI, expect this to happen mid next week.  Rationale for delays: BigInt in the driver PR was put down somewhat to focus on other pressing bugs but the CI patches ran revealed the areas we need to put logic to get this working Risks: No risks. 2023-01-20: Status update: Working on support for BSON useBigInt64 flag to Node Driver  Rationale for delays: No delays. Risks: No risks. 2023-01-06: Status update: Working on EJSON parse/stringify support for BigInt to/from $numberLong Rationale for delays: No delays. Risks: No risks. 2022-12-23: Status update: Support BigInt64 deserialization in js-bson in review. Rationale for delays: No delays. Risks: No risks. 2022-12-09: Status update: Started project, working on breakdown of project tickets Rationale for delays: No delays. Risks: No risks.
    • 6
    • 7
    • 7

      Epic Summary

      Summary
      This epic captures several tickets that have cropped up from users indicating that we need to improve how we serialize BigInts.

      Design Document

      Motivation
      Due to several users highlighting issues here, Rachelle would like the Node team to consider prioritizing this following the 5.0 compat work.

      Cast of Characters

      Engineering Lead: Jeff Yemin
      Product Owner: Rachelle Palmer
      Program Manager: Martin Bajana

            Assignee:
            warren.james@mongodb.com Warren James
            Reporter:
            alexander.golin@mongodb.com Alexander Golin (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved:
              11 weeks, 4 days