Uploaded image for project: 'WiredTiger'
  1. WiredTiger
  2. WT-10993

Don't use internal WiredTiger structures in the cache_resize.cpp test

    • Type: Icon: Technical Debt Technical Debt
    • Resolution: Unresolved
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • 5
    • StorEng - Defined Pipeline

      The test cache_resize.cpp contains the following code:

      void
          set_tracking_cursor(WT_SESSION *session, const tracking_operation &operation, const uint64_t &,
            const std::string &, const std::string &value, uint64_t ts,
            scoped_cursor &op_track_cursor) override final
          {
              uint64_t txn_id = ((WT_SESSION_IMPL *)session)->txn->id;
      

      The transaction id is then supposed to be used as part of the validation stage. However, this is not working and the fix for this test depends on WT-9339. On top of this, it is generally bad practice to rely on internal structures at the application level and it prevents WT-10965 from progressing.

      For these reasons, the logic of the test should be redesigned without the usage of internal structures.

            Assignee:
            backlog-server-storage-engines [DO NOT USE] Backlog - Storage Engines Team
            Reporter:
            etienne.petrel@mongodb.com Etienne Petrel
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: