[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: |