[SERVER-34388] Continue converting snapshot reads test to use multi-statement txn api Created: 09/Apr/18  Updated: 29/Oct/23  Resolved: 23/May/18

Status: Closed
Project: Core Server
Component/s: Storage
Affects Version/s: None
Fix Version/s: 4.0.0-rc1, 4.1.1

Type: Improvement Priority: Major - P3
Reporter: Xiangyu Yao (Inactive) Assignee: Xiangyu Yao (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Depends
depends on SERVER-34339 Aggregation error aborts the transact... Closed
depends on SERVER-34441 Error when aborting/committing a none... Closed
depends on SERVER-34288 Allow abortTransaction or commitTrans... Closed
Related
related to SERVER-35017 Multi-document transactions do not re... Closed
is related to SERVER-34094 Change snapshot read tests to use mul... Closed
Backwards Compatibility: Fully Compatible
Backport Requested:
v4.0
Sprint: Storage NYC 2018-05-21, Storage NYC 2018-06-04
Participants:

 Description   

As SERVER-34094 is currently blocked by some fixes for the multi-statement txn api, this ticket is created as the followup work. The tests need to be converted after the fixes for the api are:
afterClusterTime_committed_reads.js
agg_explain_read_concern.js
read_concern_snapshot_yielding.js
readConcern_snapshot.js
read_majority.js
readConcern_atClusterTime_snapshot_selection.js



 Comments   
Comment by Githook User [ 23/May/18 ]

Author:

{'username': 'xy24', 'name': 'Xiangyu Yao', 'email': 'xiangyu.yao@mongodb.com'}

Message: SERVER-34388 Convert tests doing snapshot reads to use transactions

(cherry picked from commit cb2e0d6e9cd588661b2ed7d22692da3171a82f3a)
Branch: v4.0
https://github.com/mongodb/mongo/commit/3b823758c1f16463b4ef8058a75c827753533ce8

Comment by Githook User [ 22/May/18 ]

Author:

{'username': 'xy24', 'name': 'Xiangyu Yao', 'email': 'xiangyu.yao@mongodb.com'}

Message: SERVER-34388 Convert tests doing snapshot reads to use transactions
Branch: master
https://github.com/mongodb/mongo/commit/cb2e0d6e9cd588661b2ed7d22692da3171a82f3a

Comment by Max Hirschhorn [ 17/May/18 ]

I filed SERVER-35017 to convert the test, since it will require bug fixes to get it to pass.

I've introduced a new jstests/replsets/dbhash_at_cluster_time.js test as part of SERVER-34778 to confirm my changes from SERVER-35017 so the jstests/noPassthrough/readConcern_atClusterTime_snapshot_selection.js still needs to be converted to use multi-statement transactions.

Comment by Tess Avitabile (Inactive) [ 16/May/18 ]

I filedĀ SERVER-35017 to convert the test, since it will require bug fixes to get it to pass.

Comment by Tess Avitabile (Inactive) [ 16/May/18 ]

Do we have plans to convert readConcern_atClusterTime_snapshot_selection.js? This is important for testing that transactions select the correct snapshot for atClusterTime.

Generated at Thu Feb 08 04:36:31 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.