[SERVER-41489] fassert the status rather than status.isOK() in KVCatalog Created: 04/Jun/19  Updated: 29/Oct/23  Resolved: 16/Aug/19

Status: Closed
Project: Core Server
Component/s: Storage
Affects Version/s: None
Fix Version/s: 4.3.1

Type: Improvement Priority: Major - P3
Reporter: Jason Chan Assignee: Daniel Solnik (Inactive)
Resolution: Fixed Votes: 0
Labels: neweng
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Backwards Compatibility: Fully Compatible
Sprint: Execution Team 2019-08-26
Participants:

 Description   

We should update the fassert here and here to assert on just the status rather than status.isOK(). The isOK() should be implied as part of the assertion and this way the status code will be logged on failure.



 Comments   
Comment by Githook User [ 16/Aug/19 ]

Author:

{'name': 'Daniel Solnik', 'email': 'dansolnik@gmail.com', 'username': 'Dsolnik'}

Message: SERVER-41489 fassert the status rather than status.isOK() in kv_catalog.cpp
Branch: master
https://github.com/mongodb/mongo/commit/b0815d97b1ff33f2977124908859411312a4eba5

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