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


##########
be/src/olap/rowset/rowset_meta.h:
##########
@@ -302,6 +302,12 @@ class RowsetMeta {
         return num_segments() > 1 && is_singleton_delta() && 
segments_overlap() != NONOVERLAPPING;
     }
 
+    bool produced_by_compaction() const {

Review Comment:
   warning: function 'produced_by_compaction' should be marked [[nodiscard]] 
[modernize-use-nodiscard]
   
   ```suggestion
       [[nodiscard]] bool produced_by_compaction() const {
   ```
   



##########
be/src/olap/rowset/rowset.h:
##########
@@ -165,6 +165,7 @@ class Rowset : public std::enable_shared_from_this<Rowset> {
     bool is_segments_overlapping() const { return 
rowset_meta()->is_segments_overlapping(); }
     KeysType keys_type() { return _schema->keys_type(); }
     RowsetStatePB rowset_meta_state() const { return 
rowset_meta()->rowset_state(); }
+    bool produced_by_compaction() const { return 
rowset_meta()->produced_by_compaction(); }

Review Comment:
   warning: function 'produced_by_compaction' should be marked [[nodiscard]] 
[modernize-use-nodiscard]
   
   ```suggestion
   ; }[[nodiscard]] 
   ```
   



-- 
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