[SERVER-61010] Make BucketCatalog::abort Status parameter non-optional Created: 26/Oct/21 Updated: 29/Oct/23 Resolved: 28/Feb/22 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.0.0-rc0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Gregory Noma | Assignee: | Gregory Noma |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Sprint: | Execution Team 2022-03-07 |
| Participants: |
| Description |
|
BucketCatalog::abort takes an optional Status which it uses as the error code to abort all batches on the same bucket. If no Status is provided, then it defaults to using TimeseriesBucketCleared. However, it would be better to require callers to specify an abort status. This way we won't potentially lose the error that caused the abort across batches, by converting to TimeseriesBucketCleared when really it was a different status that was the cause. |
| Comments |
| Comment by Githook User [ 28/Feb/22 ] |
|
Author: {'name': 'Gregory Noma', 'email': 'gregory.noma@gmail.com', 'username': 'gregorynoma'}Message: |