uchenily commented on PR #61240:
URL: https://github.com/apache/doris/pull/61240#issuecomment-4109306289
```cpp
TabletSharedPtr get_tablet(TTabletId tablet_id, bool include_deleted =
false,
std::string* err = nullptr);
TabletSharedPtr get_tablet(TTabletId tablet_id, TabletUid tablet_uid,
bool include_deleted = false, std::string*
err = nullptr);
```
The `TabletManager::get_tablet` currently returns TabletSharedPtr, every
caller has to check nullptr. Changing it to return a `Status` type would
simplify the code. But get_tablet is used in many places, before making those
changes, I want to ask for your advice.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
