This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push:
new b24ac3b51a5 branch-3.0: [fix](schema-change) Remove incorrect
assertion for tablet state after committing sc job #48219 (#48439)
b24ac3b51a5 is described below
commit b24ac3b51a512eb7553c1f20db17a13b226ed5c4
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Mar 13 19:51:06 2025 +0800
branch-3.0: [fix](schema-change) Remove incorrect assertion for tablet
state after committing sc job #48219 (#48439)
Cherry-picked from #48219
Co-authored-by: Siyang Tang <tangsiy...@selectdb.com>
---
be/src/cloud/cloud_schema_change_job.cpp | 7 -------
1 file changed, 7 deletions(-)
diff --git a/be/src/cloud/cloud_schema_change_job.cpp
b/be/src/cloud/cloud_schema_change_job.cpp
index b98dfb33a24..a109640e467 100644
--- a/be/src/cloud/cloud_schema_change_job.cpp
+++ b/be/src/cloud/cloud_schema_change_job.cpp
@@ -401,13 +401,6 @@ Status
CloudSchemaChangeJob::_convert_historical_rowsets(const SchemaChangeParam
const auto& stats = finish_resp.stats();
{
std::unique_lock wlock(_new_tablet->get_header_lock());
- // new_tablet's state MUST be `TABLET_NOTREADY`, because we won't sync
a new tablet in schema change job
- DCHECK(_new_tablet->tablet_state() == TABLET_NOTREADY);
- if (_new_tablet->tablet_state() != TABLET_NOTREADY) [[unlikely]] {
- LOG(ERROR) << "invalid tablet state, tablet_id=" <<
_new_tablet->tablet_id();
- return Status::InternalError("invalid tablet state, tablet_id={}",
- _new_tablet->tablet_id());
- }
_new_tablet->add_rowsets(std::move(_output_rowsets), true, wlock);
_new_tablet->set_cumulative_layer_point(_output_cumulative_point);
_new_tablet->reset_approximate_stats(stats.num_rowsets(),
stats.num_segments(),
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
