github-actions[bot] commented on code in PR #17542:
URL: https://github.com/apache/doris/pull/17542#discussion_r1141996344


##########
be/src/olap/rowset/beta_rowset_writer.h:
##########
@@ -132,12 +132,15 @@ class BetaRowsetWriter : public RowsetWriter {
     Status _rename_compacted_segments(int64_t begin, int64_t end);
     Status _rename_compacted_segment_plain(uint64_t seg_id);
 
+    void set_segment_start_id(int32_t start_id) { _segment_start_id = 
start_id; }

Review Comment:
   warning: 'set_segment_start_id' overrides a member function but is not 
marked 'override' [clang-diagnostic-inconsistent-missing-override]
   ```cpp
       void set_segment_start_id(int32_t start_id) { _segment_start_id = 
start_id; }
            ^
   ```
   **be/src/olap/rowset/rowset_writer.h:94:** overridden virtual function is 
here
   ```cpp
       virtual void set_segment_start_id(int num_segment) { LOG(FATAL) << "not 
supported!"; }
                    ^
   ```
   



-- 
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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to