[refactor](read-uncommitted) Register uncommitted rowset in commit_rowset

Move registration into CloudDeltaWriter::commit_rowset() right after
the meta service commit succeeds, which is the natural point where the
rowset becomes committed-but-not-published.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: dataroaring

be/src/cloud/cloud_delta_writer.cpp

@@ -121,7 +121,9 @@ Status CloudDeltaWriter::commit_rowset() {
     }
 
     // Handle normal rowset with data
-    return _engine.meta_mgr().commit_rowset(*rowset_meta(), "");
+    RETURN_IF_ERROR(_engine.meta_mgr().commit_rowset(*rowset_meta(), ""));
+    rowset_builder()->register_uncommitted_rowset();
+    return Status::OK();
 }
 
 Status CloudDeltaWriter::_commit_empty_rowset() {
@@ -146,8 +148,4 @@ Status CloudDeltaWriter::set_txn_related_delete_bitmap() {
     return rowset_builder()->set_txn_related_delete_bitmap();
 }
 
-void CloudDeltaWriter::register_uncommitted_rowset() {
-    rowset_builder()->register_uncommitted_rowset();
-}
-
 } // namespace doris

be/src/cloud/cloud_delta_writer.h

@@ -53,8 +53,6 @@ class CloudDeltaWriter final : public BaseDeltaWriter {
 
     Status set_txn_related_delete_bitmap();
 
-    void register_uncommitted_rowset();
-
     std::shared_ptr<ResourceContext> resource_context() { return _resource_ctx; }
 
 private:

be/src/cloud/cloud_tablets_channel.cpp

@@ -261,11 +261,6 @@ Status CloudTabletsChannel::close(LoadChannel* parent, const PTabletWriterAddBlo
         it++;
     }
 
-    // 7. Register uncommitted rowsets for READ UNCOMMITTED visibility
-    for (auto* writer : writers_to_commit) {
-        writer->register_uncommitted_rowset();
-    }
-
     tablet_vec->Reserve(static_cast<int>(writers_to_commit.size()));
     for (auto* writer : writers_to_commit) {
         PTabletInfo* tablet_info = tablet_vec->Add();