Re: [PR] [feature](nereids) Support SELECT * REPLACE clause [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38680:
URL: https://github.com/apache/doris/pull/38680#issuecomment-2262197319

   
   
   TPC-DS: Total hot run time: 169621 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit ab90698d95aa8f7b2569ccc70cf0e6199ba0f266, 
data reload: false
   
   query1   926 375 373 373
   query2   6474168616891686
   query3   6687216 221 216
   query4   19972   17601   17362   17362
   query5   4322535 523 523
   query6   290 194 151 151
   query7   4601298 301 298
   query8   243 198 195 195
   query9   8510235423602354
   query10  455 289 267 267
   query11  10492   999299219921
   query12  135 91  87  87
   query13  1656387 374 374
   query14  11318   795785677957
   query15  210 161 170 161
   query16  6833452 444 444
   query17  954 584 552 552
   query18  1359294 289 289
   query19  199 153 148 148
   query20  99  87  89  87
   query21  206 102 101 101
   query22  4229388139283881
   query23  33806   33257   32946   32946
   query24  10375   308230333033
   query25  712 401 400 400
   query26  1722159 156 156
   query27  2959286 277 277
   query28  6918197219581958
   query29  1103440 442 440
   query30  291 153 154 153
   query31  942 753 770 753
   query32  103 58  58  58
   query33  706 317 321 317
   query34  924 474 487 474
   query35  849 747 739 739
   query36  984 863 871 863
   query37  258 78  79  78
   query38  2868275827702758
   query39  847 800 830 800
   query40  281 116 113 113
   query41  48  48  46  46
   query42  125 105 107 105
   query43  459 432 433 432
   query44  1278724 714 714
   query45  207 173 178 173
   query46  1085806 802 802
   query47  1768169116921691
   query48  363 303 295 295
   query49  1171426 424 424
   query50  891 436 441 436
   query51  6893676067006700
   query52  109 90  99  90
   query53  254 191 176 176
   query54  638 450 454 450
   query55  79  75  75  75
   query56  288 247 273 247
   query57  1180104910571049
   query58  275 269 263 263
   query59  2659245223342334
   query60  294 270 272 270
   query61  97  97  107 97
   query62  889 663 657 657
   query63  220 192 202 192
   query64  5992191619011901
   query65  3187308330833083
   query66  1378326 332 326
   query67  15159   14815   14665   14665
   query68  4547568 579 568
   query69  486 314 300 300
   query70  1113107210111011
   query71  441 284 270 270
   query72  7762269025242524
   query73  766 328 336 328
   query74  6005561556755615
   query75  3636275427602754
   query76  2806133414191334
   query77  499 341 317 317
   query78  9403897188918891
   query79  2471562 534 534
   query80  1168500 500 500
   query81  587 226 216 216
   query82  744 130 129 129
   query83  238 169 172 169
   query84  275 79  83  79
   query85  2056303 295 295
   query86  473 326 277 277
   query87  3276310630853085
   query88  3831239424182394
   query89  391 289 287 287
   query90  1910192 190 190
   query91  126 99  99  99
   query92  70  54  55  54
   query93  2464617 610 610
   query94  921 283 294 283
   query95  390 270 277 270
   query96  612 279 280 279
   query97  3242302030693020
   query98  230 237 199 199
   query99  1698133013041304
   Total cold run time: 268441 ms
   Total hot run time: 169621 ms
   ```
   
   


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

Re: [PR] [fix](merge-on-write) segcompaction should process delete bitmap if necessary [doris]

2024-08-01 Thread via GitHub


zhannngchen merged PR #38369:
URL: https://github.com/apache/doris/pull/38369


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



(doris) branch master updated: [fix](merge-on-write) segcompaction should process delete bitmap if necessary (#38369)

2024-08-01 Thread zhangchen
This is an automated email from the ASF dual-hosted git repository.

zhangchen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
 new c03b564d6a1 [fix](merge-on-write) segcompaction should process delete 
bitmap if necessary (#38369)
c03b564d6a1 is described below

commit c03b564d6a1ae477d58d3370fb5e9f9c60e92cc2
Author: zhannngchen <48427519+zhannngc...@users.noreply.github.com>
AuthorDate: Thu Aug 1 15:00:28 2024 +0800

[fix](merge-on-write) segcompaction should process delete bitmap if 
necessary (#38369)

## Proposed changes

Issue Number: close #xxx

When loading data to a unique key table with sequence column, some data
in current load job might be marked as delete due to a lower sequence
value.
If there's many segments in such load job, segcompaction might be
triggered, which don't process the delete bitmap currently, will cause
data correctness issue

For example:
1. we have 4 segments in current load job initially, and due to seq
column, some rows are marked as deleted
2. after segcompaction, if we don't process the delete bitmap, it's
content is still corresponding to the old segment layout, and row
7,14,15 is not mark deleted correctly on new generated segment 1.
3. in this PR, we convert old delete bitmap to fit new segment layout,
it use similar way as base/cumulative compaction to convert delete
bitmaps on old layout to new one, but the rowid conversion is simpler



![whiteboard_exported_image-2](https://github.com/user-attachments/assets/a419b6a4-e583-457a-bf4e-56d9bd2a3544)
---
 be/src/olap/merger.cpp|  19 +-
 be/src/olap/merger.h  |   6 +-
 be/src/olap/rowset/beta_rowset_writer.cpp |  34 +-
 be/src/olap/rowset/beta_rowset_writer.h   |   3 +-
 be/src/olap/rowset/segcompaction.cpp  |  67 ++-
 be/src/olap/rowset/segcompaction.h|  13 +
 be/src/olap/simple_rowid_conversion.h |  84 +++
 be/src/vec/olap/vertical_block_reader.cpp |   1 +
 be/test/common/status_test.cpp|   2 -
 be/test/olap/segcompaction_mow_test.cpp   | 890 ++
 10 files changed, 1100 insertions(+), 19 deletions(-)

diff --git a/be/src/olap/merger.cpp b/be/src/olap/merger.cpp
index ad70241ad87..87792db93a6 100644
--- a/be/src/olap/merger.cpp
+++ b/be/src/olap/merger.cpp
@@ -336,16 +336,12 @@ Status Merger::vertical_compact_one_group(
 }
 
 // for segcompaction
-Status Merger::vertical_compact_one_group(int64_t tablet_id, ReaderType 
reader_type,
-  const TabletSchema& tablet_schema, 
bool is_key,
-  const std::vector& 
column_group,
-  vectorized::RowSourcesBuffer* 
row_source_buf,
-  vectorized::VerticalBlockReader& 
src_block_reader,
-  segment_v2::SegmentWriter& 
dst_segment_writer,
-  int64_t max_rows_per_segment, 
Statistics* stats_output,
-  uint64_t* index_size, KeyBoundsPB& 
key_bounds) {
-// build tablet reader
-VLOG_NOTICE << "vertical compact one group, max_rows_per_segment=" << 
max_rows_per_segment;
+Status Merger::vertical_compact_one_group(
+int64_t tablet_id, ReaderType reader_type, const TabletSchema& 
tablet_schema, bool is_key,
+const std::vector& column_group, 
vectorized::RowSourcesBuffer* row_source_buf,
+vectorized::VerticalBlockReader& src_block_reader,
+segment_v2::SegmentWriter& dst_segment_writer, Statistics* 
stats_output,
+uint64_t* index_size, KeyBoundsPB& key_bounds, SimpleRowIdConversion* 
rowid_conversion) {
 // TODO: record_rowids
 vectorized::Block block = tablet_schema.create_block(column_group);
 size_t output_rows = 0;
@@ -362,6 +358,9 @@ Status Merger::vertical_compact_one_group(int64_t 
tablet_id, ReaderType reader_t
"failed to write block when merging 
rowsets of tablet " +
std::to_string(tablet_id));
 
+if (is_key && rowid_conversion != nullptr) {
+
rowid_conversion->add(src_block_reader.current_block_row_locations());
+}
 output_rows += block.rows();
 block.clear_column_data();
 }
diff --git a/be/src/olap/merger.h b/be/src/olap/merger.h
index 7513c90fbd1..cb05162b3bc 100644
--- a/be/src/olap/merger.h
+++ b/be/src/olap/merger.h
@@ -23,6 +23,7 @@
 #include "io/io_common.h"
 #include "olap/iterators.h"
 #include "olap/rowset/rowset_fwd.h"
+#include "olap/simple_rowid_conversion.h"
 #include "olap/tablet_fwd.h"
 
 namespace doris {
@@ -82,8 +83,9 @@ public:
  vectorized::Row

Re: [PR] [Improvement](mow) Make calculate delete bitmap timeout configurable [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38668:
URL: https://github.com/apache/doris/pull/38668#issuecomment-2262198467

   PR approved by at least one committer and no changes requested.


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



[PR] [fix](fe) Fix the default value of ReplacePartitionClause.isStrictRange [doris]

2024-08-01 Thread via GitHub


w41ter opened a new pull request, #38688:
URL: https://github.com/apache/doris/pull/38688

   During ReplacePartitionClause.analyze(), the default value of isStrictRange 
is true. However, ReplacePartitionClause instances constructed by internal code 
do not call analyze(), so their isStrictRange value is incorrect (e.g., INSERT 
INTO ... OVERWRITE).
   
   This PR handle the relevant properties when constructing 
ReplacePartitionClause.
   
   
   


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



Re: [PR] [fix](fe) Fix the default value of ReplacePartitionClause.isStrictRange [doris]

2024-08-01 Thread via GitHub


w41ter commented on PR #38688:
URL: https://github.com/apache/doris/pull/38688#issuecomment-2262203279

   run buildall


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



Re: [PR] [fix](fe) Fix the default value of ReplacePartitionClause.isStrictRange [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38688:
URL: https://github.com/apache/doris/pull/38688#issuecomment-2262202968

   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your 
PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
   
   Since 2024-03-18, the Document has been moved to 
[doris-website](https://github.com/apache/doris-website).
   See [Doris 
Document](https://cwiki.apache.org/confluence/display/DORIS/Doris+Document).


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



Re: [PR] [fix](case) fix jsonb-load output [doris]

2024-08-01 Thread via GitHub


amorynan commented on PR #38661:
URL: https://github.com/apache/doris/pull/38661#issuecomment-2262205875

   run buildall


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



Re: [PR] [feature](nereids) Support SELECT * REPLACE clause [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38680:
URL: https://github.com/apache/doris/pull/38680#issuecomment-2262206044

   
   
   ClickBench: Total hot run time: 29.9 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit ab90698d95aa8f7b2569ccc70cf0e6199ba0f266, 
data reload: false
   
   query1   0.040.040.04
   query2   0.070.040.03
   query3   0.230.050.04
   query4   1.680.070.06
   query5   0.560.500.48
   query6   1.150.720.72
   query7   0.020.010.01
   query8   0.050.040.04
   query9   0.570.500.52
   query10  0.560.560.56
   query11  0.140.110.11
   query12  0.150.120.12
   query13  0.620.600.59
   query14  0.770.800.79
   query15  0.910.860.86
   query16  0.350.350.35
   query17  1.051.010.97
   query18  0.220.210.21
   query19  1.881.781.74
   query20  0.020.010.01
   query21  15.41   0.780.66
   query22  3.897.321.17
   query23  17.99   1.301.35
   query24  2.290.230.22
   query25  0.190.080.08
   query26  0.320.220.22
   query27  0.450.230.24
   query28  13.16   1.000.96
   query29  12.52   3.353.32
   query30  0.260.060.06
   query31  2.840.420.40
   query32  3.230.490.48
   query33  2.962.962.92
   query34  15.46   4.274.26
   query35  4.314.324.33
   query36  0.690.490.48
   query37  0.200.160.16
   query38  0.160.160.15
   query39  0.040.040.04
   query40  0.160.130.12
   query41  0.110.040.05
   query42  0.060.050.04
   query43  0.050.040.04
   Total cold run time: 107.79 s
   Total hot run time: 29.9 s
   ```
   
   


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



Re: [PR] [feature](nereids) Support SELECT * REPLACE clause [doris]

2024-08-01 Thread via GitHub


morrySnow commented on code in PR #38680:
URL: https://github.com/apache/doris/pull/38680#discussion_r1699544183


##
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/BindExpression.java:
##
@@ -633,7 +639,31 @@ private Plan 
bindProject(MatchingContext> ctx) {
 if (!excepts.isEmpty()) {
 slots = Utils.filterImmutableList(slots, slot -> 
!boundExcepts.get().contains(slot));
 }
-boundProjections.addAll(slots);
+if (!replaces.isEmpty()) {
+final Map replaceMap = new 
HashMap<>();
+final Set replaced = new HashSet<>();
+for (NamedExpression replace : boundReplaces.get()) {
+Preconditions.checkArgument(replace instanceof Alias);
+Alias alias = (Alias) replace;
+replaceMap.put(alias.getName(), alias);
+}
+
+for (Slot slot : slots) {
+if (replaceMap.containsKey(slot.getName())) {

Review Comment:
   what will happen if query like `select * replace ('123' as c1) from t, t t2` 
when t has columns `c1`?
   i think we should thrown exception same as `select c1 from t, t t2`;
   
   so the better way to do it may be we should use alias name as UnboundSlot, 
and use them to bind with project's child's output. and then we use 
`org.apache.doris.nereids.util.ExpressionUtils#replace` to replace project list.



##
fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java:
##
@@ -3009,10 +3012,23 @@ protected LogicalPlan withSelectQuerySpecification(
 throw new ParseException("only column name is 
supported in except clause", selectColumnCtx);
 }
 UnboundStar star = new UnboundStar(ImmutableList.of());
-project = new LogicalProject<>(ImmutableList.of(star), 
expressions, isDistinct, aggregate);
+project = new LogicalProject<>(
+ImmutableList.of(star), expressions, 
ImmutableList.of(), isDistinct, aggregate);
+} else if (selectColumnCtx.REPLACE() != null) {

Review Comment:
   `else if` not a good way. if we support use expect + replace, we should use 
`if`. if we do not support use them togather, we should throw exception.



##
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalProject.java:
##


Review Comment:
   update hashCode, equals and toString



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



Re: [PR] [enhancement](err-msg) Add detailed column and schema info when failed to create a column iterator [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38689:
URL: https://github.com/apache/doris/pull/38689#issuecomment-2262211297

   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your 
PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
   
   Since 2024-03-18, the Document has been moved to 
[doris-website](https://github.com/apache/doris-website).
   See [Doris 
Document](https://cwiki.apache.org/confluence/display/DORIS/Doris+Document).


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



Re: [PR] [improvement](statisitcs)Disable external catalog auto analyze by default. [doris]

2024-08-01 Thread via GitHub


Jibing-Li commented on PR #38674:
URL: https://github.com/apache/doris/pull/38674#issuecomment-2262211359

   run p1


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



Re: [PR] [2.0](dependencies)upgrade dependencies [doris]

2024-08-01 Thread via GitHub


CalvinKirs merged PR #38671:
URL: https://github.com/apache/doris/pull/38671


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



(doris) branch branch-2.0 updated: [2.0](dependencies)upgrade dependencies (#38671)

2024-08-01 Thread kirs
This is an automated email from the ASF dual-hosted git repository.

kirs pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-2.0 by this push:
 new aced7cf1ec6 [2.0](dependencies)upgrade dependencies (#38671)
aced7cf1ec6 is described below

commit aced7cf1ec606c0bee696dc6f020e16c223aead4
Author: Calvin Kirs 
AuthorDate: Thu Aug 1 15:08:24 2024 +0800

[2.0](dependencies)upgrade dependencies (#38671)

## Proposed changes

Issue Number: close #xxx


---
 fe/pom.xml | 18 --
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/fe/pom.xml b/fe/pom.xml
index 3cb5af10e6e..234d9fc3bcd 100644
--- a/fe/pom.xml
+++ b/fe/pom.xml
@@ -260,14 +260,15 @@ under the License.
 2.18.0
 2.0.6
 4.0.2
-4.1.94.Final
+
+4.1.104.Final
 
 3.10.6.Final
 2.1
 
 1.34.0
-1.58.0
-3.38.0
+1.60.1
+3.43.0
 3.24.3
 
 
@@ -275,7 +276,7 @@ under the License.
 
com.google.protobuf:protoc:${protoc.artifact.version}
 
io.grpc:protoc-gen-grpc-java:${grpc-java.version}
 3.1.5
-1.1.10.1
+1.1.10.5
 1.11-8
 1.0.1
 6.7.2
@@ -283,7 +284,7 @@ under the License.
 4.9.3
 3.4.0
 2.0
-1.1.0.Final
+2.0.1.Final
 0.2.3
 3.4.0
 6.4.5
@@ -342,6 +343,7 @@ under the License.
 3.0.0
 
 2.3.2
+0.27
 
 0.7.0-incubating
 3.4.4
@@ -417,7 +419,11 @@ under the License.
 opentelemetry-sdk
 ${opentelemetry.version}
 
-
+
+io.airlift
+aircompressor
+${aircompressor.version}
+
 
 io.opentelemetry
 opentelemetry-exporter-otlp


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



[PR] [enhancement](err-msg) Add detailed column and schema info when failed to create a column iterator [doris]

2024-08-01 Thread via GitHub


TangSiyang2001 opened a new pull request, #38689:
URL: https://github.com/apache/doris/pull/38689

   
   ## Proposed changes
   
   Issue Number: close #xxx
   
   
   
   


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



Re: [PR] [branch-2.1] Picks "[opt](merge-on-write) Skip the alignment process of some rowsets in partial update #38487" [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38682:
URL: https://github.com/apache/doris/pull/38682#issuecomment-2262214552

   TeamCity be ut coverage result:
Function Coverage: 36.43% (9254/25402) 
Line Coverage: 27.98% (75676/270417)
Region Coverage: 26.79% (38891/145191)
Branch Coverage: 23.51% (19735/83932)
Coverage Report: 
http://coverage.selectdb-in.cc/coverage/e38eedb85a2627e9e2342e661797354d17cb906d_e38eedb85a2627e9e2342e661797354d17cb906d/report/index.html


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



Re: [PR] [regression](kerberos)add hive with kerberos write back case [doris]

2024-08-01 Thread via GitHub


wsjz commented on PR #38647:
URL: https://github.com/apache/doris/pull/38647#issuecomment-2262217769

   run buildall


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



Re: [PR] [Fix](Json) fix some cast issue [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38683:
URL: https://github.com/apache/doris/pull/38683#issuecomment-2262220483

   
   
   TPC-H: Total hot run time: 41330 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit b06f3e3fe1ecb4d99c8304c9fd5442f2e18607ff, 
data reload: false
   
   -- Round 1 --
   q1   17695   408040094009
   q2   2017202 199 199
   q3   10461   132613131313
   q4   10164   776 892 776
   q5   7635298929422942
   q6   220 138 138 138
   q7   1019620 609 609
   q8   9435190119391901
   q9   8651662765636563
   q10  8753383438053805
   q11  434 249 253 249
   q12  431 229 230 229
   q13  17761   294629422942
   q14  268 251 247 247
   q15  523 486 493 486
   q16  525 393 385 385
   q17  954 918 895 895
   q18  7996732771937193
   q19  1488121312021202
   q20  565 330 335 330
   q21  5290464546374637
   q22  346 298 280 280
   Total cold run time: 112631 ms
   Total hot run time: 41330 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   4026399740283997
   q2   338 230 226 226
   q3   2986300630993006
   q4   1960201019721972
   q5   5609549554415441
   q6   233 130 133 130
   q7   2161180218411802
   q8   3336341433633363
   q9   8708869387888693
   q10  3939401539693969
   q11  536 453 451 451
   q12  774 564 604 564
   q13  15206   314531453145
   q14  301 270 270 270
   q15  527 507 489 489
   q16  466 421 414 414
   q17  1770173117231723
   q18  8074784476317631
   q19  1725174317061706
   q20  2067187418531853
   q21  5756553754605460
   q22  522 462 468 462
   Total cold run time: 71020 ms
   Total hot run time: 56767 ms
   ```
   
   


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



Re: [PR] [enhancement](err-msg) Add detailed column and schema info when failed to create a column iterator [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38689:
URL: https://github.com/apache/doris/pull/38689#issuecomment-226365

   clang-tidy review says "All clean, LGTM! :+1:"


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



Re: [PR] [draft](cloud-merge) Support shadow tablet to do cumulative compaction in cloud mode [doris]

2024-08-01 Thread via GitHub


Lchangliang commented on PR #37293:
URL: https://github.com/apache/doris/pull/37293#issuecomment-226622

   run buildall


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



Re: [PR] [feature](nereids) Support SELECT * REPLACE clause [doris]

2024-08-01 Thread via GitHub


macroguo-ghy commented on code in PR #38680:
URL: https://github.com/apache/doris/pull/38680#discussion_r1699559515


##
fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java:
##
@@ -3009,10 +3012,23 @@ protected LogicalPlan withSelectQuerySpecification(
 throw new ParseException("only column name is 
supported in except clause", selectColumnCtx);
 }
 UnboundStar star = new UnboundStar(ImmutableList.of());
-project = new LogicalProject<>(ImmutableList.of(star), 
expressions, isDistinct, aggregate);
+project = new LogicalProject<>(
+ImmutableList.of(star), expressions, 
ImmutableList.of(), isDistinct, aggregate);
+} else if (selectColumnCtx.REPLACE() != null) {

Review Comment:
   ok, I will fix it. I think we should support combine `REPLACE` and `EXCEPT`. 
I see ck support this syntax, 
https://clickhouse.com/docs/en/sql-reference/statements/select#modifier-combinations



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



Re: [PR] [Fix](hdfs-fs)The cache expiration should explicitly release the held fs [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38610:
URL: https://github.com/apache/doris/pull/38610#issuecomment-2262223114

   
   
   TPC-H: Total hot run time: 41306 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit d9e3536ad4eebdd65bf77a485a0e78f21fdf8d6e, 
data reload: false
   
   -- Round 1 --
   q1   17601   408840434043
   q2   2022197 197 197
   q3   10464   132513121312
   q4   10172   873 911 873
   q5   7622292229492922
   q6   220 136 137 136
   q7   1029606 592 592
   q8   9425192219291922
   q9   8673657965606560
   q10  8780384837843784
   q11  431 246 241 241
   q12  424 220 220 220
   q13  17791   290029172900
   q14  271 254 250 250
   q15  519 495 491 491
   q16  514 393 391 391
   q17  949 918 861 861
   q18  7919735072037203
   q19  1430121311981198
   q20  554 323 336 323
   q21  5258461747404617
   q22  338 270 272 270
   Total cold run time: 112406 ms
   Total hot run time: 41306 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   4019398839923988
   q2   321 225 223 223
   q3   2975297330412973
   q4   1975196820071968
   q5   5503549954245424
   q6   232 131 128 128
   q7   2098173518461735
   q8   3309338033213321
   q9   8643863786588637
   q10  3999396640613966
   q11  547 442 449 442
   q12  721 589 575 575
   q13  15761   309230823082
   q14  302 267 275 267
   q15  530 478 502 478
   q16  459 416 422 416
   q17  1755174217161716
   q18  8109774975937593
   q19  1703173217241724
   q20  2055183618111811
   q21  5669538554625385
   q22  564 470 458 458
   Total cold run time: 71249 ms
   Total hot run time: 56310 ms
   ```
   
   


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



Re: [PR] [fix](test) Modify SQLServer image to custom hub (#38515) [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38690:
URL: https://github.com/apache/doris/pull/38690#issuecomment-2262224578

   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your 
PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
   
   Since 2024-03-18, the Document has been moved to 
[doris-website](https://github.com/apache/doris-website).
   See [Doris 
Document](https://cwiki.apache.org/confluence/display/DORIS/Doris+Document).


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



Re: [PR] [Improvement](mow) Make calculate delete bitmap timeout configurable [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38668:
URL: https://github.com/apache/doris/pull/38668#issuecomment-2262223357

   
   
   TPC-H: Total hot run time: 41456 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit aff2dd6b37b68d3b9e2bb24b5806d706bfef821d, 
data reload: false
   
   -- Round 1 --
   q1   17689   407640594059
   q2   2028201 207 201
   q3   10455   124313441243
   q4   10159   788 918 788
   q5   7626288929482889
   q6   225 140 141 140
   q7   1028616 623 616
   q8   9428186319681863
   q9   8528660166026601
   q10  8758387238303830
   q11  440 255 254 254
   q12  433 234 229 229
   q13  17759   294129412941
   q14  269 239 245 239
   q15  517 489 484 484
   q16  519 400 386 386
   q17  958 924 921 921
   q18  7869723071887188
   q19  1579122612121212
   q20  543 316 315 315
   q21  5367477847924778
   q22  352 296 279 279
   Total cold run time: 112529 ms
   Total hot run time: 41456 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   4091401440254014
   q2   327 220 217 217
   q3   2982299331382993
   q4   1976201920072007
   q5   5579552554545454
   q6   231 133 138 133
   q7   2126180318571803
   q8   3325339833493349
   q9   8749866887518668
   q10  3977402239413941
   q11  548 453 452 452
   q12  763 590 629 590
   q13  16565   308831113088
   q14  309 264 298 264
   q15  539 496 489 489
   q16  476 415 407 407
   q17  1779171317091709
   q18  8073773375717571
   q19  1743170817231708
   q20  2030185218771852
   q21  5930543153965396
   q22  519 445 462 445
   Total cold run time: 72637 ms
   Total hot run time: 56550 ms
   ```
   
   


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



Re: [PR] [FIX](regresscases) fix regress exception for test_array_element_at_and_slice [doris]

2024-08-01 Thread via GitHub


amorynan commented on PR #31621:
URL: https://github.com/apache/doris/pull/31621#issuecomment-2262223948

   2.0 不用合入 这是nereids 的case


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



[PR] [fix](test) Modify SQLServer image to custom hub (#38515) [doris]

2024-08-01 Thread via GitHub


hello-stephen opened a new pull request, #38690:
URL: https://github.com/apache/doris/pull/38690

   pick from master #38515 
   ## Proposed changes
   
   Issue Number: close #xxx
   
   
   
   


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



Re: [PR] [fix](test) Modify SQLServer image to custom hub (#38515) [doris]

2024-08-01 Thread via GitHub


hello-stephen commented on PR #38690:
URL: https://github.com/apache/doris/pull/38690#issuecomment-2262226246

   run buildall


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



Re: [PR] [Fix](Json) fix some cast issue [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38683:
URL: https://github.com/apache/doris/pull/38683#issuecomment-2262239111

   
   
   TPC-DS: Total hot run time: 169501 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit b06f3e3fe1ecb4d99c8304c9fd5442f2e18607ff, 
data reload: false
   
   query1   905 371 367 367
   query2   6452163516691635
   query3   219 223 219
   query4   20428   17286   17191   17191
   query5   3633543 533 533
   query6   299 178 163 163
   query7   4601298 293 293
   query8   258 202 200 200
   query9   8495233323062306
   query10  426 278 267 267
   query11  10453   10049   98579857
   query12  122 93  87  87
   query13  1628382 373 373
   query14  8777708679457086
   query15  208 161 168 161
   query16  6756462 435 435
   query17  971 583 550 550
   query18  1880291 297 291
   query19  195 144 148 144
   query20  96  85  84  84
   query21  205 105 99  99
   query22  4229392739913927
   query23  33953   33915   33557   33557
   query24  10086   311431213114
   query25  689 453 443 443
   query26  1691168 158 158
   query27  2986285 290 285
   query28  7413203120052005
   query29  1196476 459 459
   query30  239 158 156 156
   query31  941 787 771 771
   query32  108 59  59  59
   query33  690 328 342 328
   query34  933 505 512 505
   query35  896 756 762 756
   query36  1071901 901 901
   query37  180 86  86  86
   query38  2959279228512792
   query39  892 821 835 821
   query40  258 122 126 122
   query41  48  46  49  46
   query42  122 104 98  98
   query43  482 410 416 410
   query44  1208728 718 718
   query45  208 185 183 183
   query46  1105793 797 793
   query47  1772168217031682
   query48  368 301 294 294
   query49  913 438 434 434
   query50  910 434 442 434
   query51  6790671466846684
   query52  104 92  95  92
   query53  256 184 190 184
   query54  666 460 463 460
   query55  76  80  77  77
   query56  295 269 270 269
   query57  1141103511501035
   query58  281 264 281 264
   query59  2525239423032303
   query60  292 281 286 281
   query61  103 94  95  94
   query62  908 683 674 674
   query63  213 183 176 176
   query64  5651191819301918
   query65  3151309131573091
   query66  1326342 348 342
   query67  15146   14718   15130   14718
   query68  6195589 589 589
   query69  763 391 348 348
   query70  1116101710591017
   query71  533 283 290 283
   query72  8034268324882488
   query73  995 328 326 326
   query74  6004558455195519
   query75  4388270527352705
   query76  4453135213761352
   query77  752 317 317 317
   query78  9482891089748910
   query79  2935528 533 528
   query80  2048515 508 508
   query81  558 219 217 217
   query82  1221138 135 135
   query83  269 185 174 174
   query84  264 82  81  81
   query85  1430310 314 310
   query86  438 289 282 282
   query87  3293307630933076
   query88  3704239623932393
   query89  411 288 281 281
   query90  1974192 188 188
   query91  131 101 102 101
   query92  61  51  53  51
   query93  3589619 611 611
   query94  843 303 269 269
   query95  379 268 272 268
   query96  626 283 281 281
   query97  3249304531223045
   query98  230 202 197 197
   query99  1611131113331311
   Total cold run time: 272903 ms
   Total hot run time: 169501 ms
   ```
   
   


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

Re: [PR] [fix](Nereids) remove db readlock before get table from db [doris]

2024-08-01 Thread via GitHub


morrySnow commented on PR #38660:
URL: https://github.com/apache/doris/pull/38660#issuecomment-2262241019

   run could_p0


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



Re: [PR] [Fix](hdfs-fs)The cache expiration should explicitly release the held fs [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38610:
URL: https://github.com/apache/doris/pull/38610#issuecomment-2262241750

   
   
   TPC-DS: Total hot run time: 170308 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit d9e3536ad4eebdd65bf77a485a0e78f21fdf8d6e, 
data reload: false
   
   query1   904 378 366 366
   query2   6467167716431643
   query3   6665214 229 214
   query4   20251   17564   17234   17234
   query5   3647498 504 498
   query6   284 158 157 157
   query7   4583300 285 285
   query8   248 192 189 189
   query9   8520233023152315
   query10  429 290 270 270
   query11  10655   10051   10069   10051
   query12  121 98  88  88
   query13  1645377 371 371
   query14  8859800981838009
   query15  205 162 170 162
   query16  6776440 458 440
   query17  965 569 545 545
   query18  1882310 286 286
   query19  196 147 151 147
   query20  91  88  87  87
   query21  211 104 102 102
   query22  4428424740554055
   query23  33665   33610   33434   33434
   query24  9311308632203086
   query25  662 385 413 385
   query26  1438150 154 150
   query27  3007285 277 277
   query28  7546201019991999
   query29  1073441 430 430
   query30  237 156 154 154
   query31  939 752 755 752
   query32  102 56  58  56
   query33  677 298 314 298
   query34  925 492 553 492
   query35  851 780 770 770
   query36  1042886 874 874
   query37  172 81  81  81
   query38  3058283928682839
   query39  919 813 828 813
   query40  252 111 109 109
   query41  47  43  47  43
   query42  115 97  96  96
   query43  484 399 420 399
   query44  1165722 716 716
   query45  206 183 179 179
   query46  1098783 771 771
   query47  1795170717421707
   query48  355 286 285 285
   query49  871 411 405 405
   query50  911 422 433 422
   query51  6844661766056605
   query52  99  91  87  87
   query53  250 175 175 175
   query54  620 449 432 432
   query55  77  74  75  74
   query56  278 256 247 247
   query57  1126105010351035
   query58  267 265 278 265
   query59  25052362
   query60  324 275 274 274
   query61  97  94  95  94
   query62  865 679 665 665
   query63  215 181 185 181
   query64  5589192218671867
   query65  3150308130793079
   query66  1003322 336 322
   query67  15256   14898   14889   14889
   query68  5761568 574 568
   query69  705 395 301 301
   query70  1134104910641049
   query71  465 272 266 266
   query72  7682269125182518
   query73  914 326 333 326
   query74  6045567857565678
   query75  3585274527332733
   query76  3292138614151386
   query77  572 307 302 302
   query78  9461890890508908
   query79  2146535 535 535
   query80  2074496 513 496
   query81  567 227 218 218
   query82  821 131 127 127
   query83  291 171 166 166
   query84  268 80  76  76
   query85  1442315 303 303
   query86  470 317 297 297
   query87  3235306731333067
   query88  3715236923822369
   query89  384 287 282 282
   query90  1767189 187 187
   query91  121 97  102 97
   query92  59  49  48  48
   query93  2379616 612 612
   query94  767 285 295 285
   query95  390 286 263 263
   query96  607 284 278 278
   query97  3244304430823044
   query98  227 201 199 199
   query99  1702127512761275
   Total cold run time: 265812 ms
   Total hot run time: 170308 ms
   ```
   
   


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

Re: [PR] [Improvement](mow) Make calculate delete bitmap timeout configurable [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38668:
URL: https://github.com/apache/doris/pull/38668#issuecomment-2262242029

   
   
   TPC-DS: Total hot run time: 170667 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit aff2dd6b37b68d3b9e2bb24b5806d706bfef821d, 
data reload: false
   
   query1   914 371 371 371
   query2   6479167816421642
   query3   6673218 222 218
   query4   19745   17496   17399   17399
   query5   3667518 539 518
   query6   278 180 174 174
   query7   4589307 312 307
   query8   252 197 208 197
   query9   8507236523252325
   query10  429 284 264 264
   query11  10424   995599309930
   query12  117 88  89  88
   query13  1619379 368 368
   query14  9649790485587904
   query15  196 159 165 159
   query16  6816441 457 441
   query17  935 559 539 539
   query18  1897271 279 271
   query19  183 143 143 143
   query20  93  85  85  85
   query21  199 104 106 104
   query22  4130400538873887
   query23  33951   33707   33260   33260
   query24  9117316331053105
   query25  661 392 433 392
   query26  1402165 156 156
   query27  2988282 296 282
   query28  7434202320152015
   query29  1015441 446 441
   query30  236 160 164 160
   query31  954 787 800 787
   query32  107 56  59  56
   query33  687 334 330 330
   query34  906 510 515 510
   query35  887 757 778 757
   query36  1088873 913 873
   query37  168 82  83  82
   query38  2997281328422813
   query39  933 817 808 808
   query40  251 111 115 111
   query41  47  42  45  42
   query42  115 108 102 102
   query43  474 418 430 418
   query44  1207723 736 723
   query45  203 180 180 180
   query46  1120831 798 798
   query47  1805169817171698
   query48  359 285 292 285
   query49  854 418 432 418
   query50  907 440 439 439
   query51  6765667666196619
   query52  99  91  89  89
   query53  261 186 185 185
   query54  603 466 447 447
   query55  78  79  75  75
   query56  281 261 247 247
   query57  1145103010121012
   query58  262 265 294 265
   query59  2633249223152315
   query60  289 279 280 279
   query61  96  90  92  90
   query62  883 660 666 660
   query63  213 195 187 187
   query64  4858203620092009
   query65  3133312931013101
   query66  1257346 340 340
   query67  15310   15060   14950   14950
   query68  5341600 610 600
   query69  776 420 319 319
   query70  1132107810351035
   query71  430 283 279 279
   query72  7730281126072607
   query73  922 329 335 329
   query74  6010562956495629
   query75  3428275727622757
   query76  3068141414291414
   query77  571 320 314 314
   query78  9553892888528852
   query79  2272552 543 543
   query80  987 508 516 508
   query81  548 233 220 220
   query82  1430127 131 127
   query83  201 171 174 171
   query84  258 79  80  79
   query85  1413323 297 297
   query86  514 289 289 289
   query87  3297306930973069
   query88  3706240624132406
   query89  397 295 291 291
   query90  1759187 193 187
   query91  127 99  101 99
   query92  58  48  50  48
   query93  2090626 621 621
   query94  780 279 296 279
   query95  378 267 266 266
   query96  614 283 278 278
   query97  3207303030703030
   query98  223 202 206 202
   query99  1645127712861277
   Total cold run time: 263695 ms
   Total hot run time: 170667 ms
   ```
   
   


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

[PR] [fix](parquet) disable parquet page index by default [doris]

2024-08-01 Thread via GitHub


morningman opened a new pull request, #38691:
URL: https://github.com/apache/doris/pull/38691

   there are some issue with page index parsing.
   disable it temporarily


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



Re: [PR] [fix](parquet) disable parquet page index by default [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38691:
URL: https://github.com/apache/doris/pull/38691#issuecomment-2262244720

   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your 
PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
   
   Since 2024-03-18, the Document has been moved to 
[doris-website](https://github.com/apache/doris-website).
   See [Doris 
Document](https://cwiki.apache.org/confluence/display/DORIS/Doris+Document).


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



Re: [PR] [fix](jsonb)fix jsonb deserialze [doris]

2024-08-01 Thread via GitHub


amorynan commented on PR #38629:
URL: https://github.com/apache/doris/pull/38629#issuecomment-2262247297

   run p0


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



Re: [PR] [fix](syntax) multi statements multi delim with semicolon [doris]

2024-08-01 Thread via GitHub


morrySnow commented on PR #38670:
URL: https://github.com/apache/doris/pull/38670#issuecomment-2262247622

   run buildall


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



Re: [PR] [Fix](Json) fix some cast issue [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38683:
URL: https://github.com/apache/doris/pull/38683#issuecomment-2262248651

   
   
   ClickBench: Total hot run time: 30.2 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit b06f3e3fe1ecb4d99c8304c9fd5442f2e18607ff, 
data reload: false
   
   query1   0.040.040.04
   query2   0.070.040.03
   query3   0.220.050.05
   query4   1.690.070.07
   query5   0.470.480.48
   query6   1.140.720.71
   query7   0.020.020.01
   query8   0.050.050.04
   query9   0.570.510.50
   query10  0.580.560.57
   query11  0.160.120.11
   query12  0.140.120.12
   query13  0.600.590.60
   query14  0.770.800.80
   query15  0.900.860.85
   query16  0.350.360.37
   query17  1.010.970.98
   query18  0.210.210.21
   query19  1.881.721.73
   query20  0.010.010.01
   query21  15.46   0.790.67
   query22  4.347.061.54
   query23  17.83   1.381.31
   query24  2.270.230.21
   query25  0.180.080.08
   query26  0.320.210.21
   query27  0.450.230.23
   query28  13.16   1.010.96
   query29  12.55   3.363.33
   query30  0.250.060.06
   query31  2.870.400.41
   query32  3.240.490.48
   query33  2.932.922.95
   query34  15.44   4.254.24
   query35  4.264.274.32
   query36  0.680.470.48
   query37  0.200.170.16
   query38  0.160.140.16
   query39  0.040.040.03
   query40  0.150.130.14
   query41  0.100.040.04
   query42  0.050.040.05
   query43  0.050.050.05
   Total cold run time: 107.86 s
   Total hot run time: 30.2 s
   ```
   
   


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



Re: [PR] [fix](parquet) disable parquet page index by default [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38691:
URL: https://github.com/apache/doris/pull/38691#issuecomment-2262249855

   PR approved by anyone and no changes requested.


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



Re: [PR] [Improvement](mow) Make calculate delete bitmap timeout configurable [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38668:
URL: https://github.com/apache/doris/pull/38668#issuecomment-2262251550

   
   
   ClickBench: Total hot run time: 30 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit aff2dd6b37b68d3b9e2bb24b5806d706bfef821d, 
data reload: false
   
   query1   0.040.040.03
   query2   0.070.040.04
   query3   0.220.050.05
   query4   1.690.070.06
   query5   0.480.490.49
   query6   1.170.710.71
   query7   0.030.020.01
   query8   0.050.040.04
   query9   0.570.520.51
   query10  0.580.560.56
   query11  0.160.120.12
   query12  0.150.130.13
   query13  0.610.600.60
   query14  0.770.790.81
   query15  0.880.860.88
   query16  0.350.360.35
   query17  0.990.980.99
   query18  0.220.210.20
   query19  1.831.741.72
   query20  0.010.010.01
   query21  15.40   0.770.65
   query22  4.127.481.32
   query23  17.98   1.311.24
   query24  2.260.220.22
   query25  0.180.070.08
   query26  0.320.220.22
   query27  0.460.230.22
   query28  13.16   0.980.98
   query29  12.52   3.313.34
   query30  0.270.060.05
   query31  2.860.420.40
   query32  3.770.500.49
   query33  2.992.962.92
   query34  15.45   4.264.25
   query35  4.314.324.31
   query36  0.670.470.48
   query37  0.210.160.16
   query38  0.170.160.16
   query39  0.040.040.04
   query40  0.160.130.12
   query41  0.100.050.05
   query42  0.060.050.05
   query43  0.050.040.04
   Total cold run time: 108.38 s
   Total hot run time: 30 s
   ```
   
   


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



Re: [PR] [fix](fe) Fix the default value of ReplacePartitionClause.isStrictRange [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38688:
URL: https://github.com/apache/doris/pull/38688#issuecomment-2262249769

   
   
   TPC-H: Total hot run time: 41905 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit 6bc3da05d1601eb79b9f899867eeca81e7238910, 
data reload: false
   
   -- Round 1 --
   q1   18889   427241314131
   q2   2021210 208 208
   q3   10444   132313131313
   q4   10171   875 912 875
   q5   7673302229922992
   q6   226 142 142 142
   q7   1052618 615 615
   q8   9447194519891945
   q9   8514660166126601
   q10  8717380838753808
   q11  427 257 260 257
   q12  410 242 234 234
   q13  17760   294129462941
   q14  272 247 239 239
   q15  538 493 486 486
   q16  493 392 388 388
   q17  966 935 947 935
   q18  7968743272277227
   q19  1380121912211219
   q20  564 329 338 329
   q21  5328478847384738
   q22  363 286 282 282
   Total cold run time: 113623 ms
   Total hot run time: 41905 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   4064404240364036
   q2   329 230 220 220
   q3   2994299329952993
   q4   1892188918651865
   q5   5292526252505250
   q6   221 133 134 133
   q7   2065167516821675
   q8   3248329332893289
   q9   8349832783138313
   q10  3753386638103810
   q11  547 457 443 443
   q12  720 567 557 557
   q13  17013   296229562956
   q14  289 256 253 253
   q15  517 485 481 481
   q16  433 402 394 394
   q17  1718169716941694
   q18  7790732971827182
   q19  1675168716751675
   q20  1981177217601760
   q21  5445523751895189
   q22  511 477 473 473
   Total cold run time: 70846 ms
   Total hot run time: 54641 ms
   ```
   
   


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



Re: [PR] [Fix](hdfs-fs)The cache expiration should explicitly release the held fs [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38610:
URL: https://github.com/apache/doris/pull/38610#issuecomment-2262251271

   
   
   ClickBench: Total hot run time: 29.81 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit d9e3536ad4eebdd65bf77a485a0e78f21fdf8d6e, 
data reload: false
   
   query1   0.040.040.03
   query2   0.070.040.04
   query3   0.230.050.05
   query4   1.670.060.07
   query5   0.480.470.48
   query6   1.130.710.71
   query7   0.020.010.01
   query8   0.060.040.04
   query9   0.570.510.51
   query10  0.560.590.55
   query11  0.160.110.11
   query12  0.150.120.13
   query13  0.610.600.60
   query14  0.770.800.79
   query15  0.890.850.85
   query16  0.360.360.35
   query17  1.031.001.01
   query18  0.220.210.21
   query19  1.811.761.75
   query20  0.020.020.01
   query21  15.39   0.770.65
   query22  3.888.591.12
   query23  18.02   1.321.28
   query24  2.270.220.21
   query25  0.180.090.08
   query26  0.310.210.22
   query27  0.460.230.22
   query28  13.16   0.990.96
   query29  12.55   3.323.32
   query30  0.250.060.05
   query31  2.870.410.39
   query32  3.260.480.48
   query33  2.922.962.97
   query34  15.45   4.264.27
   query35  4.304.304.29
   query36  0.680.480.49
   query37  0.180.160.16
   query38  0.160.150.15
   query39  0.040.030.03
   query40  0.160.130.13
   query41  0.090.050.04
   query42  0.050.040.04
   query43  0.050.040.04
   Total cold run time: 107.53 s
   Total hot run time: 29.81 s
   ```
   
   


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



Re: [PR] [fix](parquet) disable parquet page index by default [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38691:
URL: https://github.com/apache/doris/pull/38691#issuecomment-2262255831

   clang-tidy review says "All clean, LGTM! :+1:"


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



Re: [PR] [fix](session) fix select * from variables system table [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #34529:
URL: https://github.com/apache/doris/pull/34529#issuecomment-2262257070

   PR approved by at least one committer and no changes requested.


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



Re: [PR] [fix](session) fix select * from variables system table [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #34529:
URL: https://github.com/apache/doris/pull/34529#issuecomment-2262257113

   PR approved by anyone and no changes requested.


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



Re: [PR] [bugfix](iceberg)revert count(*) directly returned by fe [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38566:
URL: https://github.com/apache/doris/pull/38566#issuecomment-2262262774

   
   
   TPC-H: Total hot run time: 42008 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit 3e7cf484bc9cc7925d70aee8591b20167fe3, 
data reload: false
   
   -- Round 1 --
   q1   17637   410340954095
   q2   2026201 207 201
   q3   10448   129313591293
   q4   10182   826 916 826
   q5   7676302429702970
   q6   223 138 136 136
   q7   1044621 626 621
   q8   9460193419651934
   q9   8514665266626652
   q10  8773387338693869
   q11  438 256 264 256
   q12  447 231 233 231
   q13  17755   294129432941
   q14  277 245 246 245
   q15  521 491 504 491
   q16  526 387 385 385
   q17  974 940 920 920
   q18  7968731773147314
   q19  1390123412211221
   q20  558 321 327 321
   q21  5385482348024802
   q22  352 284 294 284
   Total cold run time: 112574 ms
   Total hot run time: 42008 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   4042404540394039
   q2   333 225 220 220
   q3   3006300031443000
   q4   2017206219881988
   q5   5635549154485448
   q6   220 132 132 132
   q7   2129177918141779
   q8   3328341633853385
   q9   8785866387788663
   q10  4000407739983998
   q11  555 453 460 453
   q12  788 596 617 596
   q13  14740   316631513151
   q14  302 272 286 272
   q15  533 483 478 478
   q16  469 433 424 424
   q17  1778175417391739
   q18  8334769077657690
   q19  1734172117561721
   q20  2068183018831830
   q21  5810553252765276
   q22  561 470 457 457
   Total cold run time: 71167 ms
   Total hot run time: 56739 ms
   ```
   
   


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



Re: [PR] [feature](recycler) Support recycling data stored in different version path formats across different types of storage vaults [doris]

2024-08-01 Thread via GitHub


platoneko commented on PR #38415:
URL: https://github.com/apache/doris/pull/38415#issuecomment-2262263854

   run buildall


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



Re: [PR] [fix](fe) Fix the default value of ReplacePartitionClause.isStrictRange [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38688:
URL: https://github.com/apache/doris/pull/38688#issuecomment-2262268663

   
   
   TPC-DS: Total hot run time: 169238 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit 6bc3da05d1601eb79b9f899867eeca81e7238910, 
data reload: false
   
   query1   909 375 366 366
   query2   6466171616341634
   query3   6679222 232 222
   query4   20241   17413   17577   17413
   query5   4305517 529 517
   query6   283 175 164 164
   query7   4605302 300 300
   query8   259 194 196 194
   query9   8530237423532353
   query10  465 289 269 269
   query11  10522   10021   10174   10021
   query12  147 87  89  87
   query13  1635376 377 376
   query14  9574797572287228
   query15  227 165 170 165
   query16  7117471 494 471
   query17  954 574 557 557
   query18  1906298 282 282
   query19  213 148 152 148
   query20  104 89  87  87
   query21  203 102 100 100
   query22  4242392838813881
   query23  33757   32802   32834   32802
   query24  10349   312430313031
   query25  706 404 413 404
   query26  1789154 153 153
   query27  2994284 286 284
   query28  6979197719621962
   query29  1366445 433 433
   query30  285 153 152 152
   query31  937 775 759 759
   query32  104 57  59  57
   query33  711 323 333 323
   query34  899 488 487 487
   query35  844 746 732 732
   query36  1003875 888 875
   query37  302 82  81  81
   query38  2889275627672756
   query39  870 805 814 805
   query40  297 129 118 118
   query41  48  47  50  47
   query42  120 102 103 102
   query43  452 411 425 411
   query44  1195731 727 727
   query45  209 178 180 178
   query46  1102825 792 792
   query47  1798170516901690
   query48  367 302 294 294
   query49  1192456 454 454
   query50  905 444 445 444
   query51  6855665266416641
   query52  101 96  93  93
   query53  259 187 185 185
   query54  646 463 460 460
   query55  80  84  81  81
   query56  304 276 263 263
   query57  1167105510301030
   query58  309 282 275 275
   query59  2694241225932412
   query60  311 280 316 280
   query61  119 113 114 113
   query62  1061670 649 649
   query63  220 189 193 189
   query64  5823192518791879
   query65  3271311230923092
   query66  1429337 332 332
   query67  15317   14763   14652   14652
   query68  6888589 593 589
   query69  742 371 304 304
   query70  1096104711241047
   query71  574 282 286 282
   query72  8020268025022502
   query73  981 329 337 329
   query74  6006565956765659
   query75  4498275527062706
   query76  4939140814251408
   query77  764 328 322 322
   query78  9526898288308830
   query79  3716542 545 542
   query80  1115528 498 498
   query81  576 229 230 229
   query82  778 138 133 133
   query83  345 182 177 177
   query84  275 80  88  80
   query85  1545322 305 305
   query86  443 336 300 300
   query87  3283311631093109
   query88  3776243324222422
   query89  459 289 293 289
   query90  2107198 197 197
   query91  129 105 101 101
   query92  64  53  50  50
   query93  4505616 616 616
   query94  971 282 293 282
   query95  391 263 275 263
   query96  623 283 280 280
   query97  3235306431083064
   query98  210 203 195 195
   query99  1631131613141314
   Total cold run time: 278232 ms
   Total hot run time: 169238 ms
   ```
   
   


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

Re: [I] [Bug] RoutineLoad重复消费 [doris]

2024-08-01 Thread via GitHub


dataroaring commented on issue #38551:
URL: https://github.com/apache/doris/issues/38551#issuecomment-2262273360

   You can set offsets when creating a new job, it consums from the specified 
offsets.


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



Re: [PR] Revert "[feat](Nereids): Optimize Query Plan by Pulling Up Join with Common Child from Union (#36397)" [doris]

2024-08-01 Thread via GitHub


morrySnow merged PR #38561:
URL: https://github.com/apache/doris/pull/38561


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



(doris) branch master updated (c03b564d6a1 -> 2e1219a7f56)

2024-08-01 Thread morrysnow
This is an automated email from the ASF dual-hosted git repository.

morrysnow pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


from c03b564d6a1 [fix](merge-on-write) segcompaction should process delete 
bitmap if necessary (#38369)
 add 2e1219a7f56 Revert "[feat](Nereids): Optimize Query Plan by Pulling Up 
Join with Common Child from Union (#36397)" (#38561)

No new revisions were added by this update.

Summary of changes:
 .../doris/nereids/jobs/executor/Rewriter.java  |   4 +-
 .../org/apache/doris/nereids/rules/RuleType.java   |   1 -
 .../nereids/rules/rewrite/PullUpJoinFromUnion.java | 484 -
 .../rules/rewrite/PullUpJoinFromUnionTest.java | 171 
 .../data/nereids_hint_tpcds_p0/shape/query14.out   |  61 +--
 .../data/nereids_hint_tpcds_p0/shape/query71.out   |  41 +-
 .../data/nereids_hint_tpcds_p0/shape/query76.out   |  54 ++-
 .../join_pull_up_union/join_pull_up_union.out  | 118 -
 .../shape/query14.out  |  61 +--
 .../shape/query71.out  |  45 +-
 .../shape/query76.out  |  54 ++-
 .../noStatsRfPrune/query14.out |  49 ++-
 .../noStatsRfPrune/query71.out |  41 +-
 .../noStatsRfPrune/query76.out |  51 ++-
 .../no_stats_shape/query14.out |  63 +--
 .../no_stats_shape/query71.out |  41 +-
 .../no_stats_shape/query76.out |  51 ++-
 .../rf_prune/query14.out   |  49 ++-
 .../rf_prune/query71.out   |  45 +-
 .../rf_prune/query76.out   |  63 +--
 .../nereids_tpcds_shape_sf100_p0/shape/query14.out |  61 +--
 .../nereids_tpcds_shape_sf100_p0/shape/query71.out |  45 +-
 .../nereids_tpcds_shape_sf100_p0/shape/query76.out |  63 +--
 .../tpcds_sf100/noStatsRfPrune/query14.out |  53 ++-
 .../tpcds_sf100/noStatsRfPrune/query71.out |  41 +-
 .../tpcds_sf100/noStatsRfPrune/query76.out |  63 +--
 .../tpcds_sf100/no_stats_shape/query14.out |  59 +--
 .../tpcds_sf100/no_stats_shape/query71.out |  41 +-
 .../tpcds_sf100/no_stats_shape/query76.out |  63 +--
 .../new_shapes_p0/tpcds_sf100/rf_prune/query14.out |  49 ++-
 .../new_shapes_p0/tpcds_sf100/rf_prune/query71.out |  45 +-
 .../new_shapes_p0/tpcds_sf100/rf_prune/query76.out |  63 +--
 .../new_shapes_p0/tpcds_sf100/shape/query14.out|  55 ++-
 .../new_shapes_p0/tpcds_sf100/shape/query71.out|  45 +-
 .../new_shapes_p0/tpcds_sf100/shape/query76.out|  63 +--
 .../new_shapes_p0/tpcds_sf1000/shape/query14.out   |  61 +--
 .../new_shapes_p0/tpcds_sf1000/shape/query71.out   |  45 +-
 .../new_shapes_p0/tpcds_sf1000/shape/query76.out   |  63 +--
 .../join_pull_up_union/join_pull_up_union.groovy   | 194 -
 39 files changed, 1011 insertions(+), 1708 deletions(-)
 delete mode 100644 
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/PullUpJoinFromUnion.java
 delete mode 100644 
fe/fe-core/src/test/java/org/apache/doris/nereids/rules/rewrite/PullUpJoinFromUnionTest.java
 delete mode 100644 
regression-test/data/nereids_rules_p0/join_pull_up_union/join_pull_up_union.out
 delete mode 100644 
regression-test/suites/nereids_rules_p0/join_pull_up_union/join_pull_up_union.groovy


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



Re: [PR] [bugfix](asan core) should use weak ptr in rpc context to release rf context during query context deconstructed [doris]

2024-08-01 Thread via GitHub


yiguolei merged PR #38653:
URL: https://github.com/apache/doris/pull/38653


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



(doris) branch master updated: [bugfix](asan core) should use weak ptr in rpc context to release rf context during query context deconstructed (#38653)

2024-08-01 Thread yiguolei
This is an automated email from the ASF dual-hosted git repository.

yiguolei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
 new 7e8415fe5bb [bugfix](asan core) should use weak ptr in rpc context to 
release rf context during query context deconstructed (#38653)
7e8415fe5bb is described below

commit 7e8415fe5bb0521fa3511d80a6a20cd75a1ecda6
Author: yiguolei <676222...@qq.com>
AuthorDate: Thu Aug 1 15:45:36 2024 +0800

[bugfix](asan core) should use weak ptr in rpc context to release rf 
context during query context deconstructed (#38653)

## Proposed changes

Issue Number: close #xxx
F20240731 10:38:53.012670 20850 mem_tracker_limiter.cpp:135] mem tracker
label: Query#Id=7539c7d0087442b7-a1cda6392053548a, consumption: 52, peak
consumption: 1155332, mem tracker not equal to 0 when mem tracker
destruct, this usually means that memory tracking is inaccurate and
SCOPED_ATTACH_TASK and SCOPED_SWITCH_THREAD_MEM_TRACKER_LIMITER are not
used correctly. 1. For query and load, memory leaks may have occurred,
it is expected that the query mem tracker will be bound to the thread
context using SCOPED_ATTACH_TASK and
SCOPED_SWITCH_THREAD_MEM_TRACKER_LIMITER before all memory alloc and
free. 2. If a memory alloc is recorded by this tracker, it is expected
that be recorded in this tracker when memory is freed. 3. Merge the
remaining memory tracking value by this tracker into Orphan, if you
observe that Orphan is not equal to 0 in the mem tracker web or log,
this indicates that there may be a memory leak. 4. If you need to
transfer memory tracking value between two trackers, can use
transfer_to..[Address Sanitizer]:
0x606002867d80, size 52, strack trace:
0# Allocator::alloc_impl(unsigned long, unsigned long)
1# void* phmap::priv::Allocate<4ul, doris::vectorized::Allocator_
>(doris::vectorized::Allocator_*, unsigned long)
2# phmap::priv::raw_hash_set, phmap::Hash, phmap::EqualTo,
doris::vectorized::Allocator_ >::initialize_slots(unsigned long)
3# phmap::priv::raw_hash_set, phmap::Hash, phmap::EqualTo,
doris::vectorized::Allocator_ >::resize(unsigned long)
4# phmap::priv::raw_hash_set, phmap::Hash, phmap::EqualTo,
doris::vectorized::Allocator_ >::prepare_insert(unsigned long)
5# std::pair phmap::priv::raw_hash_set, phmap::Hash, phmap::EqualTo,
doris::vectorized::Allocator_ >::find_or_prepare_insert(int const&,
unsigned long)
6# std::pair, phmap::Hash, phmap::EqualTo, doris::vectorized::Allocator_
>::iterator, bool> phmap::priv::raw_hash_set, phmap::Hash,
phmap::EqualTo, doris::vectorized::Allocator_
>::emplace_decomposable(int const&, unsigned long, int const&)
7# std::pair, phmap::Hash, phmap::EqualTo, doris::vectorized::Allocator_
>::iterator, bool> phmap::priv::raw_hash_set, phmap::Hash,
phmap::EqualTo, doris::vectorized::Allocator_
>::EmplaceDecomposable::operator()(int const&, int const&) const
8# doris::HybridSet<(doris::PrimitiveType)5, doris::DynamicContainer,
doris::vectorized::ColumnVector >::insert(void const*)
9# doris::HybridSetBase::insert(doris::HybridSetBase*)
10# doris::RuntimePredicateWrapper::merge(doris::RuntimePredicateWrapper
const*)
11# doris::IRuntimeFilter::merge_from(doris::RuntimePredicateWrapper
const*)
12#
doris::RuntimeFilterMergeControllerEntity::merge(doris::PMergeFilterRequest
const*, butil::IOBufAsZeroCopyInputStream*)
13# doris::FragmentMgr::merge_filter(doris::PMergeFilterRequest const*,
butil::IOBufAsZeroCopyInputStream*)
14# std::_Function_handler::_M_invoke(std::_Any_data const&)
15# doris::WorkThreadPool::work_thread(int)
16# execute_native_thread_routine
17# ?
18# ?

Co-authored-by: yiguolei 
---
 be/src/exprs/runtime_filter.cpp | 13 +++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/be/src/exprs/runtime_filter.cpp b/be/src/exprs/runtime_filter.cpp
index 901af0472f0..d7fc2ff7490 100644
--- a/be/src/exprs/runtime_filter.cpp
+++ b/be/src/exprs/runtime_filter.cpp
@@ -1005,7 +1005,10 @@ Status IRuntimeFilter::publish(bool publish_local) {
 class SyncSizeClosure : public AutoReleaseClosure> {
 std::shared_ptr _dependency;
-RuntimeFilterContextSPtr _rf_context;
+// Should use weak ptr here, because when query context deconstructs, 
should also delete runtime filter
+// context, it not the memory is not released. And rpc is in another 
thread, it will hold rf context
+// after query context because the rpc is not returned.
+std::weak_ptr _rf_context;
 std::string _rf_debug_info;
 using Base =
 AutoReleaseClosure>;
@@ -1021,7 +1024,13 @@ class SyncSizeClosure : public 
AutoReleaseClosuresub();
 if (status.is()) {
 // rf merger backend may fini

Re: [PR] [fix](fe) Fix the default value of ReplacePartitionClause.isStrictRange [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38688:
URL: https://github.com/apache/doris/pull/38688#issuecomment-2262278448

   
   
   ClickBench: Total hot run time: 30.03 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit 6bc3da05d1601eb79b9f899867eeca81e7238910, 
data reload: false
   
   query1   0.040.040.03
   query2   0.080.040.04
   query3   0.230.060.05
   query4   1.660.080.07
   query5   0.490.480.48
   query6   1.120.720.71
   query7   0.020.010.02
   query8   0.060.050.04
   query9   0.570.500.52
   query10  0.570.570.56
   query11  0.160.120.12
   query12  0.140.130.13
   query13  0.620.590.61
   query14  0.780.800.80
   query15  0.910.870.87
   query16  0.370.360.35
   query17  1.001.011.00
   query18  0.220.210.21
   query19  1.841.791.78
   query20  0.010.010.01
   query21  15.42   0.750.65
   query22  4.027.511.14
   query23  17.81   1.451.32
   query24  2.290.220.23
   query25  0.190.090.08
   query26  0.330.220.22
   query27  0.460.230.23
   query28  13.15   1.000.97
   query29  12.50   3.363.36
   query30  0.260.060.06
   query31  2.840.410.41
   query32  3.230.500.48
   query33  2.962.982.96
   query34  15.49   4.264.24
   query35  4.314.284.28
   query36  0.680.480.47
   query37  0.190.170.16
   query38  0.170.150.14
   query39  0.050.040.03
   query40  0.160.120.12
   query41  0.100.050.06
   query42  0.060.050.05
   query43  0.050.040.04
   Total cold run time: 107.61 s
   Total hot run time: 30.03 s
   ```
   
   


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



Re: [PR] [bugfix](iceberg)revert count(*) directly returned by fe [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38566:
URL: https://github.com/apache/doris/pull/38566#issuecomment-2262281926

   
   
   TPC-DS: Total hot run time: 170524 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit 3e7cf484bc9cc7925d70aee8591b20167fe3, 
data reload: false
   
   query1   916 384 372 372
   query2   6470169216921692
   query3   6658236 227 227
   query4   19756   17363   17434   17363
   query5   3633552 539 539
   query6   300 185 179 179
   query7   4610318 310 310
   query8   256 199 202 199
   query9   8545238223572357
   query10  416 292 283 283
   query11  10471   10169   99659965
   query12  119 91  89  89
   query13  1636370 375 370
   query14  10465   839077927792
   query15  198 165 164 164
   query16  6945505 471 471
   query17  932 563 562 562
   query18  1943297 293 293
   query19  199 157 153 153
   query20  96  91  89  89
   query21  206 102 107 102
   query22  4360394839553948
   query23  34154   33912   33382   33382
   query24  10379   312330533053
   query25  684 461 405 405
   query26  1729159 156 156
   query27  3095287 293 287
   query28  7431201319981998
   query29  1255451 436 436
   query30  239 172 159 159
   query31  966 803 798 798
   query32  110 56  60  56
   query33  701 332 344 332
   query34  935 498 529 498
   query35  875 799 754 754
   query36  1047849 873 849
   query37  287 79  82  79
   query38  2899276427472747
   query39  882 836 831 831
   query40  265 117 117 117
   query41  47  42  49  42
   query42  126 112 107 107
   query43  487 402 423 402
   query44  1202732 740 732
   query45  215 185 184 184
   query46  1103830 798 798
   query47  1800168216941682
   query48  370 295 293 293
   query49  948 444 431 431
   query50  915 440 440 440
   query51  6772664766006600
   query52  107 89  91  89
   query53  265 183 181 181
   query54  649 467 448 448
   query55  78  77  76  76
   query56  280 260 267 260
   query57  1125102310351023
   query58  277 326 284 284
   query59  2556222624212226
   query60  297 283 280 280
   query61  100 94  92  92
   query62  886 680 655 655
   query63  216 189 194 189
   query64  5631189418881888
   query65  3184310630903090
   query66  1319337 330 330
   query67  15419   14754   14693   14693
   query68  4293582 588 582
   query69  757 445 320 320
   query70  1101104010571040
   query71  447 286 282 282
   query72  7488268325412541
   query73  803 341 335 335
   query74  6052572456655665
   query75  3422276227402740
   query76  2842136214671362
   query77  577 315 319 315
   query78  9363893989348934
   query79  1709536 541 536
   query80  959 525 527 525
   query81  552 225 227 225
   query82  843 136 139 136
   query83  255 177 176 176
   query84  263 83  81  81
   query85  1089312 304 304
   query86  393 321 297 297
   query87  3280311431643114
   query88  2963248824142414
   query89  384 299 292 292
   query90  1699199 193 193
   query91  134 117 103 103
   query92  60  51  53  51
   query93  1340618 622 618
   query94  764 295 267 267
   query95  384 273 277 273
   query96  606 280 278 278
   query97  3225304130393039
   query98  216 205 204 204
   query99  1610131113121311
   Total cold run time: 263310 ms
   Total hot run time: 170524 ms
   ```
   
   


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

[PR] [feature](StorageVault) Support setting path format for storage vault [doris]

2024-08-01 Thread via GitHub


platoneko opened a new pull request, #38692:
URL: https://github.com/apache/doris/pull/38692

   ## Proposed changes
   
   Support setting path format for storage vault
   


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



Re: [PR] [feature](StorageVault) Support setting path format for storage vault [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38692:
URL: https://github.com/apache/doris/pull/38692#issuecomment-2262282739

   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your 
PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
   
   Since 2024-03-18, the Document has been moved to 
[doris-website](https://github.com/apache/doris-website).
   See [Doris 
Document](https://cwiki.apache.org/confluence/display/DORIS/Doris+Document).


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



Re: [PR] [fix](syntax) multi statements must delim with semicolon [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38670:
URL: https://github.com/apache/doris/pull/38670#issuecomment-2262283557

   
   
   TPC-H: Total hot run time: 41642 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit eb64fa02dbec7a14a5784f47fefd96e4c3134e47, 
data reload: false
   
   -- Round 1 --
   q1   18441   426241684168
   q2   2486206 204 204
   q3   12293   132713791327
   q4   10747   837 957 837
   q5   7773303729762976
   q6   222 135 135 135
   q7   1040639 609 609
   q8   9443177219511772
   q9   8469660266346602
   q10  8708383438553834
   q11  431 252 246 246
   q12  410 226 235 226
   q13  17764   294629532946
   q14  275 244 245 244
   q15  523 484 502 484
   q16  496 410 391 391
   q17  976 898 873 873
   q18  8164737772417241
   q19  1391121611991199
   q20  564 337 338 337
   q21  5291474247104710
   q22  353 281 289 281
   Total cold run time: 116260 ms
   Total hot run time: 41642 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   4115405640274027
   q2   328 225 220 220
   q3   2983304230083008
   q4   1913186418621862
   q5   5260521452175214
   q6   216 133 130 130
   q7   2055166817001668
   q8   321033053305
   q9   8344831982788278
   q10  3724382238413822
   q11  545 464 458 458
   q12  728 564 529 529
   q13  13683   295929692959
   q14  290 261 253 253
   q15  517 477 478 477
   q16  459 414 408 408
   q17  1740169816971697
   q18  7745732872867286
   q19  1662168216731673
   q20  2007175617481748
   q21  5509517151195119
   q22  539 460 469 460
   Total cold run time: 67572 ms
   Total hot run time: 54601 ms
   ```
   
   


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



[PR] [Doc]Update lakehouse-overview.md [doris-website]

2024-08-01 Thread via GitHub


zhangm365 opened a new pull request, #934:
URL: https://github.com/apache/doris-website/pull/934

   修改湖仓一体方案中文版本的表达逻辑。


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



Re: [PR] [improvement](statisitcs)Disable external catalog auto analyze by default. [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38674:
URL: https://github.com/apache/doris/pull/38674#issuecomment-2262285804

   PR approved by at least one committer and no changes requested.


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



Re: [PR] [improvement](statisitcs)Disable external catalog auto analyze by default. [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38674:
URL: https://github.com/apache/doris/pull/38674#issuecomment-2262285868

   PR approved by anyone and no changes requested.


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



Re: [PR] [Feature](Cloud) Support session variable disable_file_cache in query. [doris]

2024-08-01 Thread via GitHub


wangshuo128 commented on PR #37141:
URL: https://github.com/apache/doris/pull/37141#issuecomment-2262287226

   run feut:


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



Re: [PR] [Feature](Cloud) Support session variable disable_file_cache in query. [doris]

2024-08-01 Thread via GitHub


wangshuo128 commented on PR #37141:
URL: https://github.com/apache/doris/pull/37141#issuecomment-2262287380

   run feut


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



Re: [PR] [Fix](Export) Export delete multiple times when specify the `delete_existing_files` property [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38400:
URL: https://github.com/apache/doris/pull/38400#issuecomment-2262291269

   clang-tidy review says "All clean, LGTM! :+1:"


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



Re: [PR] [bugfix](iceberg)revert count(*) directly returned by fe [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38566:
URL: https://github.com/apache/doris/pull/38566#issuecomment-2262292220

   
   
   ClickBench: Total hot run time: 29.87 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit 3e7cf484bc9cc7925d70aee8591b20167fe3, 
data reload: false
   
   query1   0.050.040.04
   query2   0.070.040.03
   query3   0.230.040.05
   query4   1.670.070.08
   query5   0.480.480.48
   query6   1.130.730.71
   query7   0.020.010.02
   query8   0.050.040.04
   query9   0.570.520.51
   query10  0.580.570.56
   query11  0.160.110.12
   query12  0.150.130.12
   query13  0.610.600.60
   query14  0.790.790.80
   query15  0.900.860.85
   query16  0.350.360.35
   query17  1.000.970.98
   query18  0.220.200.22
   query19  1.811.751.77
   query20  0.010.010.01
   query21  15.39   0.770.66
   query22  4.558.471.16
   query23  17.99   1.271.28
   query24  2.300.220.21
   query25  0.180.080.09
   query26  0.320.200.21
   query27  0.460.230.23
   query28  13.17   1.000.96
   query29  12.53   3.363.32
   query30  0.270.060.05
   query31  2.870.400.40
   query32  3.250.480.48
   query33  2.922.992.96
   query34  15.45   4.294.29
   query35  4.304.294.27
   query36  0.680.490.49
   query37  0.190.160.17
   query38  0.160.150.14
   query39  0.040.030.04
   query40  0.160.140.14
   query41  0.090.050.04
   query42  0.060.050.05
   query43  0.050.040.05
   Total cold run time: 108.23 s
   Total hot run time: 29.87 s
   ```
   
   


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



Re: [PR] [feature](StorageVault) Support setting path format for storage vault [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38692:
URL: https://github.com/apache/doris/pull/38692#issuecomment-2262295242

   clang-tidy review says "All clean, LGTM! :+1:"


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



Re: [PR] [fix](syntax) multi statements must delim with semicolon [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38670:
URL: https://github.com/apache/doris/pull/38670#issuecomment-2262303200

   
   
   TPC-DS: Total hot run time: 169176 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit eb64fa02dbec7a14a5784f47fefd96e4c3134e47, 
data reload: false
   
   query1   923 380 369 369
   query2   6491168316671667
   query3   6675218 228 218
   query4   20342   17481   17187   17187
   query5   4335506 523 506
   query6   288 168 175 168
   query7   4600311 283 283
   query8   261 196 197 196
   query9   8540236223512351
   query10  433 280 284 280
   query11  10426   987610052   9876
   query12  134 95  90  90
   query13  1641378 383 378
   query14  9415863678907890
   query15  217 162 165 162
   query16  7028475 418 418
   query17  971 564 550 550
   query18  1763296 282 282
   query19  208 147 151 147
   query20  92  85  87  85
   query21  207 103 103 103
   query22  4159408839013901
   query23  33871   33009   32955   32955
   query24  10332   304530113011
   query25  727 404 410 404
   query26  1763150 151 150
   query27  2972277 288 277
   query28  6897195619461946
   query29  1254432 422 422
   query30  291 149 151 149
   query31  960 766 753 753
   query32  101 57  57  57
   query33  710 315 395 315
   query34  913 480 500 480
   query35  831 726 721 721
   query36  996 838 858 838
   query37  285 79  91  79
   query38  2829276927352735
   query39  879 800 814 800
   query40  279 112 110 110
   query41  47  46  43  43
   query42  117 103 100 100
   query43  477 404 418 404
   query44  1194714 743 714
   query45  207 182 176 176
   query46  1082828 802 802
   query47  1756169017061690
   query48  368 290 289 289
   query49  1193439 441 439
   query50  902 432 461 432
   query51  6835668966776677
   query52  100 93  88  88
   query53  259 186 178 178
   query54  643 463 464 463
   query55  78  77  78  77
   query56  283 256 263 256
   query57  1160102610301026
   query58  277 291 263 263
   query59  2700243124982431
   query60  289 269 265 265
   query61  102 97  92  92
   query62  932 672 659 659
   query63  215 185 193 185
   query64  5918193018721872
   query65  3151307230693069
   query66  1431340 334 334
   query67  15246   14745   14935   14745
   query68  4471574 590 574
   query69  511 288 305 288
   query70  1130105110761051
   query71  446 274 274 274
   query72  8006267524892489
   query73  766 331 328 328
   query74  6038566455795579
   query75  3807274427302730
   query76  2710132113841321
   query77  688 324 300 300
   query78  9534884788588847
   query79  2695537 528 528
   query80  1066511 507 507
   query81  555 230 227 227
   query82  778 131 128 128
   query83  327 171 171 171
   query84  271 77  77  77
   query85  1891362 299 299
   query86  470 282 322 282
   query87  3244313031083108
   query88  3896241623892389
   query89  390 298 287 287
   query90  1986195 197 195
   query91  126 100 100 100
   query92  69  53  53  53
   query93  2275617 613 613
   query94  1000293 299 293
   query95  380 265 268 265
   query96  600 280 279 279
   query97  3235301430463014
   query98  216 192 200 192
   query99  1651131013121310
   Total cold run time: 268229 ms
   Total hot run time: 169176 ms
   ```
   
   


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

(doris) branch branch-2.1 updated: [Improvement](schema scan) Use async scanner for schema scanners (#38… (#38666)

2024-08-01 Thread gabriellee
This is an automated email from the ASF dual-hosted git repository.

gabriellee pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-2.1 by this push:
 new 9d23ccf1f2a [Improvement](schema scan) Use async scanner for schema 
scanners (#38… (#38666)
9d23ccf1f2a is described below

commit 9d23ccf1f2ae3fd6c3f3adc7af428c8d116f13db
Author: Gabriel 
AuthorDate: Thu Aug 1 16:05:24 2024 +0800

[Improvement](schema scan) Use async scanner for schema scanners (#38… 
(#38666)

…403)
---
 be/src/exec/schema_scanner.cpp | 69 +-
 be/src/exec/schema_scanner.h   | 24 +++-
 .../schema_active_queries_scanner.cpp  |  2 +-
 .../schema_scanner/schema_active_queries_scanner.h |  2 +-
 .../schema_scanner/schema_backend_active_tasks.cpp |  3 +-
 .../schema_scanner/schema_backend_active_tasks.h   |  2 +-
 .../schema_scanner/schema_charsets_scanner.cpp |  2 +-
 .../exec/schema_scanner/schema_charsets_scanner.h  |  2 +-
 .../schema_scanner/schema_collations_scanner.cpp   |  2 +-
 .../schema_scanner/schema_collations_scanner.h |  2 +-
 .../exec/schema_scanner/schema_columns_scanner.cpp |  2 +-
 .../exec/schema_scanner/schema_columns_scanner.h   |  2 +-
 .../exec/schema_scanner/schema_dummy_scanner.cpp   |  2 +-
 be/src/exec/schema_scanner/schema_dummy_scanner.h  |  2 +-
 .../exec/schema_scanner/schema_files_scanner.cpp   |  2 +-
 be/src/exec/schema_scanner/schema_files_scanner.h  |  2 +-
 .../schema_metadata_name_ids_scanner.cpp   |  2 +-
 .../schema_metadata_name_ids_scanner.h |  2 +-
 .../schema_scanner/schema_partitions_scanner.cpp   |  2 +-
 .../schema_scanner/schema_partitions_scanner.h |  2 +-
 .../schema_scanner/schema_processlist_scanner.cpp  |  2 +-
 .../schema_scanner/schema_processlist_scanner.h|  2 +-
 .../schema_scanner/schema_profiling_scanner.cpp|  2 +-
 .../exec/schema_scanner/schema_profiling_scanner.h |  2 +-
 .../exec/schema_scanner/schema_routine_scanner.cpp |  2 +-
 .../exec/schema_scanner/schema_routine_scanner.h   |  2 +-
 .../exec/schema_scanner/schema_rowsets_scanner.cpp |  2 +-
 .../exec/schema_scanner/schema_rowsets_scanner.h   |  2 +-
 .../schema_schema_privileges_scanner.cpp   |  2 +-
 .../schema_schema_privileges_scanner.h |  2 +-
 .../schema_scanner/schema_schemata_scanner.cpp |  2 +-
 .../exec/schema_scanner/schema_schemata_scanner.h  |  2 +-
 .../schema_table_privileges_scanner.cpp|  2 +-
 .../schema_table_privileges_scanner.h  |  2 +-
 .../exec/schema_scanner/schema_tables_scanner.cpp  |  2 +-
 be/src/exec/schema_scanner/schema_tables_scanner.h |  2 +-
 .../schema_user_privileges_scanner.cpp |  2 +-
 .../schema_user_privileges_scanner.h   |  2 +-
 be/src/exec/schema_scanner/schema_user_scanner.cpp |  2 +-
 be/src/exec/schema_scanner/schema_user_scanner.h   |  2 +-
 .../schema_scanner/schema_variables_scanner.cpp|  2 +-
 .../exec/schema_scanner/schema_variables_scanner.h |  2 +-
 .../exec/schema_scanner/schema_views_scanner.cpp   |  2 +-
 be/src/exec/schema_scanner/schema_views_scanner.h  |  2 +-
 .../schema_workload_groups_scanner.cpp |  2 +-
 .../schema_workload_groups_scanner.h   |  2 +-
 .../schema_workload_sched_policy_scanner.cpp   |  3 +-
 .../schema_workload_sched_policy_scanner.h |  2 +-
 be/src/pipeline/exec/schema_scan_operator.cpp  |  9 ++-
 be/src/pipeline/exec/schema_scan_operator.h| 14 -
 be/src/pipeline/pipeline_x/dependency.h| 16 +
 be/src/pipeline/pipeline_x/pipeline_x_task.cpp |  2 -
 be/src/vec/exec/vschema_scan_node.cpp  |  2 +-
 53 files changed, 162 insertions(+), 68 deletions(-)

diff --git a/be/src/exec/schema_scanner.cpp b/be/src/exec/schema_scanner.cpp
index 2b6b2c1f3c0..d9cafcf9049 100644
--- a/be/src/exec/schema_scanner.cpp
+++ b/be/src/exec/schema_scanner.cpp
@@ -50,7 +50,10 @@
 #include "exec/schema_scanner/schema_workload_groups_scanner.h"
 #include "exec/schema_scanner/schema_workload_sched_policy_scanner.h"
 #include "olap/hll.h"
+#include "pipeline/pipeline_x/dependency.h"
 #include "runtime/define_primitive_type.h"
+#include "runtime/fragment_mgr.h"
+#include "runtime/types.h"
 #include "util/string_util.h"
 #include "util/types.h"
 #include "vec/columns/column.h"
@@ -64,6 +67,7 @@
 #include "vec/core/column_with_type_and_name.h"
 #include "vec/core/types.h"
 #include "vec/data_types/data_type.h"
+#include "vec/data_types/data_type_factory.hpp"
 
 namespace doris {
 class ObjectPool;
@@ -84,7 +88,60 @@ Status SchemaScanner::start(RuntimeState* state) {
 return Status::OK();
 }
 
-Status SchemaScanner::get_next_block(vectorized::Block* block, bool* eos) {
+Status SchemaScanner::get_next_block(RuntimeState* state, vectorized::Block* 
block, bool* eos) {
+if (_data_block == 

Re: [PR] [fix](local exchange) Fix local exchange blocked by a huge data block [doris]

2024-08-01 Thread via GitHub


Gabriel39 merged PR #38657:
URL: https://github.com/apache/doris/pull/38657


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



Re: [PR] [feature](cloud) Add lazy commit mechanism for `commit_txn` [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38243:
URL: https://github.com/apache/doris/pull/38243#issuecomment-2262300969

   
   
   TPC-H: Total hot run time: 41825 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit f40d3af909e3a4ce446ee5217827a6811bce34d1, 
data reload: false
   
   -- Round 1 --
   q1   17655   440740714071
   q2   2028208 202 202
   q3   10546   137913731373
   q4   10237   818 935 818
   q5   7634302229762976
   q6   219 135 140 135
   q7   1039612 606 606
   q8   9443188119881881
   q9   8530662966216621
   q10  8797390438923892
   q11  435 249 239 239
   q12  412 230 225 225
   q13  17774   296929452945
   q14  269 250 247 247
   q15  517 479 484 479
   q16  518 380 393 380
   q17  998 889 918 889
   q18  8073728673557286
   q19  1490122212201220
   q20  549 341 342 341
   q21  5317472447984724
   q22  349 282 275 275
   Total cold run time: 112829 ms
   Total hot run time: 41825 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   4136403240024002
   q2   324 218 216 216
   q3   2992304431853044
   q4   2028203920062006
   q5   5679547054735470
   q6   220 130 130 130
   q7   2187185918191819
   q8   3341339033723372
   q9   8688871187828711
   q10  3962413639333933
   q11  546 455 476 455
   q12  781 601 580 580
   q13  16313   315731323132
   q14  299 277 276 276
   q15  530 497 487 487
   q16  470 415 440 415
   q17  1755174817371737
   q18  8350777977267726
   q19  1740170717101707
   q20  2054189418281828
   q21  5857546854195419
   q22  531 450 475 450
   Total cold run time: 72783 ms
   Total hot run time: 56915 ms
   ```
   
   


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



[PR] [doc](routine load) update routine load doc [doris-website]

2024-08-01 Thread via GitHub


sollhui opened a new pull request, #935:
URL: https://github.com/apache/doris-website/pull/935

   (no comment)


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



(doris) branch branch-2.1 updated (9d23ccf1f2a -> cafcf7acc1f)

2024-08-01 Thread dataroaring
This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a change to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git


from 9d23ccf1f2a [Improvement](schema scan) Use async scanner for schema 
scanners (#38… (#38666)
 add cafcf7acc1f [cherry-pick](SSL) Fix ssl connection close 2.1 (#38587) 
(#38677)

No new revisions were added by this update.

Summary of changes:
 fe/fe-core/src/main/java/org/apache/doris/mysql/MysqlChannel.java| 2 +-
 .../src/main/java/org/apache/doris/qe/MysqlConnectProcessor.java | 5 +
 2 files changed, 6 insertions(+), 1 deletion(-)


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



Re: [PR] [fix](syntax) multi statements must delim with semicolon [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38670:
URL: https://github.com/apache/doris/pull/38670#issuecomment-2262313846

   
   
   ClickBench: Total hot run time: 30.29 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit eb64fa02dbec7a14a5784f47fefd96e4c3134e47, 
data reload: false
   
   query1   0.040.030.03
   query2   0.070.040.04
   query3   0.230.050.05
   query4   1.680.060.06
   query5   0.480.490.48
   query6   1.160.720.72
   query7   0.020.010.01
   query8   0.050.040.04
   query9   0.570.510.50
   query10  0.570.550.58
   query11  0.160.110.11
   query12  0.140.130.13
   query13  0.620.610.60
   query14  0.780.800.80
   query15  0.900.860.86
   query16  0.350.340.35
   query17  1.000.990.99
   query18  0.220.210.21
   query19  1.881.751.80
   query20  0.010.010.01
   query21  15.42   0.770.65
   query22  3.847.181.48
   query23  18.04   1.361.38
   query24  2.320.210.22
   query25  0.180.070.08
   query26  0.320.220.22
   query27  0.460.240.24
   query28  13.16   1.000.97
   query29  12.56   3.393.35
   query30  0.260.060.06
   query31  2.860.410.41
   query32  3.230.490.48
   query33  2.943.022.94
   query34  15.45   4.244.26
   query35  4.294.264.28
   query36  0.670.480.47
   query37  0.200.160.16
   query38  0.160.150.15
   query39  0.040.030.04
   query40  0.150.120.14
   query41  0.110.050.05
   query42  0.060.050.05
   query43  0.050.040.04
   Total cold run time: 107.7 s
   Total hot run time: 30.29 s
   ```
   
   


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



Re: [I] [Bug] RoutineLoad重复消费 [doris]

2024-08-01 Thread via GitHub


CoderXionghs commented on issue #38551:
URL: https://github.com/apache/doris/issues/38551#issuecomment-2262305660

   > You can set offsets when creating a new job, it consums from the specified 
offsets.
  thks  for  your  answer;
  my topic has  many partitions(≥50),so set  the  offsets  will be  very  
difficult;  what  i  hope  is can  routine  load work like  other  normal  
kafka  client,when  auto.offset.reset=earliest,it  consume  from the  last  
offset,if  the  last offset  not  exist, it  consume  from  the  beginning.
   


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



Re: [PR] [doc](routine load) update routine load doc [doris-website]

2024-08-01 Thread via GitHub


sollhui commented on PR #935:
URL: https://github.com/apache/doris-website/pull/935#issuecomment-2262308673

   run buildall


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



Re: [PR] [Improvement](schema scan) Use async scanner for schema scanners (#38… [doris]

2024-08-01 Thread via GitHub


Gabriel39 merged PR #38666:
URL: https://github.com/apache/doris/pull/38666


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



(doris) branch master updated: [fix](local exchange) Fix local exchange blocked by a huge data block (#38657)

2024-08-01 Thread gabriellee
This is an automated email from the ASF dual-hosted git repository.

gabriellee pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
 new bb1a9c98d42 [fix](local exchange) Fix local exchange blocked by a huge 
data block (#38657)
bb1a9c98d42 is described below

commit bb1a9c98d42c224fefb1140158a7c52ec651c1c3
Author: Gabriel 
AuthorDate: Thu Aug 1 16:05:53 2024 +0800

[fix](local exchange) Fix local exchange blocked by a huge data block 
(#38657)

If a huge block is push into local exchanger, it will be blocked due to
concurrent problems. This PR use a unique lock to resolve it .
---
 .../local_exchange/local_exchange_sink_operator.h  |   2 +
 .../local_exchange_source_operator.cpp |   6 +-
 .../local_exchange_source_operator.h   |   2 +
 be/src/pipeline/local_exchange/local_exchanger.cpp | 115 +
 be/src/pipeline/local_exchange/local_exchanger.h   |  14 +++
 5 files changed, 72 insertions(+), 67 deletions(-)

diff --git a/be/src/pipeline/local_exchange/local_exchange_sink_operator.h 
b/be/src/pipeline/local_exchange/local_exchange_sink_operator.h
index 0ff1df26001..faa48d209f4 100644
--- a/be/src/pipeline/local_exchange/local_exchange_sink_operator.h
+++ b/be/src/pipeline/local_exchange/local_exchange_sink_operator.h
@@ -56,6 +56,8 @@ private:
 friend class PassToOneExchanger;
 friend class LocalMergeSortExchanger;
 friend class AdaptivePassthroughExchanger;
+template 
+friend class Exchanger;
 
 ExchangerBase* _exchanger = nullptr;
 
diff --git a/be/src/pipeline/local_exchange/local_exchange_source_operator.cpp 
b/be/src/pipeline/local_exchange/local_exchange_source_operator.cpp
index 56f0a157cde..6b0cca2d71a 100644
--- a/be/src/pipeline/local_exchange/local_exchange_source_operator.cpp
+++ b/be/src/pipeline/local_exchange/local_exchange_source_operator.cpp
@@ -75,11 +75,13 @@ std::string LocalExchangeSourceLocalState::debug_string(int 
indentation_level) c
 fmt::memory_buffer debug_string_buffer;
 fmt::format_to(debug_string_buffer,
"{}, _channel_id: {}, _num_partitions: {}, _num_senders: 
{}, _num_sources: {}, "
-   "_running_sink_operators: {}, _running_source_operators: 
{}, mem_usage: {}",
+   "_running_sink_operators: {}, _running_source_operators: 
{}, mem_usage: {}, "
+   "data queue info: {}",
Base::debug_string(indentation_level), _channel_id, 
_exchanger->_num_partitions,
_exchanger->_num_senders, _exchanger->_num_sources,
_exchanger->_running_sink_operators, 
_exchanger->_running_source_operators,
-   _shared_state->mem_usage.load());
+   _shared_state->mem_usage.load(),
+   _exchanger->data_queue_debug_string(_channel_id));
 size_t i = 0;
 fmt::format_to(debug_string_buffer, ", MemTrackers: ");
 for (auto* mem_tracker : _shared_state->mem_trackers) {
diff --git a/be/src/pipeline/local_exchange/local_exchange_source_operator.h 
b/be/src/pipeline/local_exchange/local_exchange_source_operator.h
index f9fa4cfa4ed..d2f68d4ebac 100644
--- a/be/src/pipeline/local_exchange/local_exchange_source_operator.h
+++ b/be/src/pipeline/local_exchange/local_exchange_source_operator.h
@@ -51,6 +51,8 @@ private:
 friend class PassToOneExchanger;
 friend class LocalMergeSortExchanger;
 friend class AdaptivePassthroughExchanger;
+template 
+friend class Exchanger;
 
 ExchangerBase* _exchanger = nullptr;
 int _channel_id;
diff --git a/be/src/pipeline/local_exchange/local_exchanger.cpp 
b/be/src/pipeline/local_exchange/local_exchanger.cpp
index 27b7fc7e7fd..647ddcfba2d 100644
--- a/be/src/pipeline/local_exchange/local_exchanger.cpp
+++ b/be/src/pipeline/local_exchange/local_exchanger.cpp
@@ -26,6 +26,37 @@
 
 namespace doris::pipeline {
 
+template 
+bool Exchanger::_enqueue_data_and_set_ready(int channel_id,
+   
LocalExchangeSinkLocalState& local_state,
+   BlockType&& block) {
+std::unique_lock l(_m);
+if (_data_queue[channel_id].enqueue(std::move(block))) {
+local_state._shared_state->set_ready_to_read(channel_id);
+return true;
+}
+return false;
+}
+
+template 
+bool Exchanger::_dequeue_data(LocalExchangeSourceLocalState& 
local_state,
+ BlockType& block, bool* eos) {
+bool all_finished = _running_sink_operators == 0;
+if (_data_queue[local_state._channel_id].try_dequeue(block)) {
+return true;
+} else if (all_finished) {
+*eos = true;
+} else {
+std::unique_lock l(_m);
+if (_data_queue[local_state._channel_id].try_dequeue(block)) {
+return true;
+}
+  

Re: [PR] [feature](http action)Add http action to show nested inverted index file (#38272) [doris]

2024-08-01 Thread via GitHub


qidaye commented on PR #38672:
URL: https://github.com/apache/doris/pull/38672#issuecomment-2262315753

   run buildall


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



Re: [PR] [cherry-pick](SSL) Fix ssl connection close 2.1 (#38587) [doris]

2024-08-01 Thread via GitHub


dataroaring merged PR #38677:
URL: https://github.com/apache/doris/pull/38677


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



Re: [PR] [fix](local exchange) Fix local exchange blocked by a huge data block… [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38693:
URL: https://github.com/apache/doris/pull/38693#issuecomment-2262317607

   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your 
PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
   
   Since 2024-03-18, the Document has been moved to 
[doris-website](https://github.com/apache/doris-website).
   See [Doris 
Document](https://cwiki.apache.org/confluence/display/DORIS/Doris+Document).


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



[PR] [fix](local exchange) Fix local exchange blocked by a huge data block… [doris]

2024-08-01 Thread via GitHub


Gabriel39 opened a new pull request, #38693:
URL: https://github.com/apache/doris/pull/38693

   … (#38657)
   
   If a huge block is push into local exchanger, it will be blocked due to 
concurrent problems. This PR use a unique lock to resolve it .
   
   ## Proposed changes
   
   Issue Number: close #xxx
   
   
   
   


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



Re: [PR] [fix](local exchange) Fix local exchange blocked by a huge data block… [doris]

2024-08-01 Thread via GitHub


Gabriel39 commented on PR #38693:
URL: https://github.com/apache/doris/pull/38693#issuecomment-2262317910

   run buildall


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



Re: [PR] [fix](case) fix jsonb-load output [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38661:
URL: https://github.com/apache/doris/pull/38661#issuecomment-2262318766

   PR approved by at least one committer and no changes requested.


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



Re: [PR] [fix](case) fix jsonb-load output [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38661:
URL: https://github.com/apache/doris/pull/38661#issuecomment-2262318833

   PR approved by anyone and no changes requested.


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



Re: [PR] [fix](mtmv) Choose a valid partition column when there are both valid and invalid expressions (#38367) [doris]

2024-08-01 Thread via GitHub


liutang123 commented on PR #38684:
URL: https://github.com/apache/doris/pull/38684#issuecomment-2262319628

   run p0


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



Re: [PR] [feature](cloud) Add lazy commit mechanism for `commit_txn` [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38243:
URL: https://github.com/apache/doris/pull/38243#issuecomment-2262321636

   
   
   TPC-DS: Total hot run time: 170685 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit f40d3af909e3a4ce446ee5217827a6811bce34d1, 
data reload: false
   
   query1   919 381 370 370
   query2   6455170816701670
   query3   6649224 221 221
   query4   20265   17544   17285   17285
   query5   3664517 520 517
   query6   284 172 178 172
   query7   4598305 305 305
   query8   274 201 205 201
   query9   8506236923562356
   query10  434 282 278 278
   query11  10519   10016   10112   10016
   query12  122 88  97  88
   query13  1646384 366 366
   query14  8846788978577857
   query15  210 199 163 163
   query16  6672462 423 423
   query17  943 586 542 542
   query18  1888287 277 277
   query19  194 142 159 142
   query20  94  85  84  84
   query21  204 104 99  99
   query22  4261412439283928
   query23  34031   33736   33739   33736
   query24  10288   314631293129
   query25  679 415 403 403
   query26  1706154 157 154
   query27  3160282 285 282
   query28  7380202519971997
   query29  1187441 447 441
   query30  240 155 154 154
   query31  955 791 808 791
   query32  102 59  57  57
   query33  672 328 329 328
   query34  913 497 503 497
   query35  855 736 773 736
   query36  1054890 891 890
   query37  272 83  81  81
   query38  2987284927812781
   query39  861 805 855 805
   query40  249 118 111 111
   query41  49  45  45  45
   query42  130 97  100 97
   query43  469 434 414 414
   query44  1167712 744 712
   query45  215 185 180 180
   query46  1102823 768 768
   query47  1816173917111711
   query48  360 298 287 287
   query49  924 420 403 403
   query50  910 429 424 424
   query51  6840680266786678
   query52  95  88  88  88
   query53  251 181 184 181
   query54  621 456 441 441
   query55  77  74  74  74
   query56  273 266 258 258
   query57  1124107610411041
   query58  278 257 299 257
   query59  2764238222812281
   query60  302 261 275 261
   query61  96  94  94  94
   query62  887 667 677 667
   query63  211 188 186 186
   query64  5712190218991899
   query65  3193309730803080
   query66  1321331 334 331
   query67  15391   14910   14715   14715
   query68  5484560 575 560
   query69  701 379 307 307
   query70  1109109610441044
   query71  468 282 272 272
   query72  7551273225042504
   query73  929 333 326 326
   query74  6115573756975697
   query75  3620273527252725
   query76  3217136713951367
   query77  538 301 302 301
   query78  9488888388868883
   query79  1686533 528 528
   query80  1908507 491 491
   query81  570 225 225 225
   query82  478 132 127 127
   query83  275 172 165 165
   query84  277 77  77  77
   query85  1162355 426 355
   query86  448 308 309 308
   query87  3321316831333133
   query88  3691238423862384
   query89  379 284 291 284
   query90  1688186 186 186
   query91  125 99  99  99
   query92  65  51  50  50
   query93  2016621 632 621
   query94  807 286 297 286
   query95  381 270 266 266
   query96  595 280 276 276
   query97  3207311530593059
   query98  246 204 196 196
   query99  1628129112771277
   Total cold run time: 265989 ms
   Total hot run time: 170685 ms
   ```
   
   


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

Re: [PR] [COMPILE](macos) do not patch bitshuffle on macos [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38634:
URL: https://github.com/apache/doris/pull/38634#issuecomment-2262324716

    `sh-checker report`
   
   To get the full details, please check in the 
[job]("https://github.com/apache/doris/actions/runs/10194559858";) output.
   
   
   shellcheck errors
   
   ```
   
   'shellcheck ' returned error 1 finding the following syntactical issues:
   
   --
   
   In thirdparty/build-thirdparty.sh line 1169:
   if [[ "${MACHINE_OS}" != "Darwin" ]] && ([[ "${arch}" == "avx2" ]] 
|| [[ "${arch}" == "avx512" ]] || [[ "${arch}" == "neon" ]]); then
   ^-- SC2235 (style): Use { 
..; } instead of (..) to avoid subshell overhead.
   
   For more information:
 https://www.shellcheck.net/wiki/SC2235 -- Use { ..; } instead of (..) to 
av...
   --
   
   You can address the above issues in one of three ways:
   1. Manually correct the issue in the offending shell script;
   2. Disable specific issues by adding the comment:
 # shellcheck disable=
   above the line that contains the issue, where  is the error code;
   3. Add '-e ' to the SHELLCHECK_OPTS setting in your .yml action file.
   
   
   
   ```
   
   
   
   shfmt errors
   
   ```
   'shfmt ' found no issues.
   
   ```
   
   
   
   


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



Re: [PR] [COMPILE](macos) do not patch bitshuffle on macos [doris]

2024-08-01 Thread via GitHub


Mryange commented on PR #38634:
URL: https://github.com/apache/doris/pull/38634#issuecomment-2262323527

   run buildall


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



Re: [PR] [pipeline](fix) Fix blocking task which is not triggered by 2nd RPC (… [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38694:
URL: https://github.com/apache/doris/pull/38694#issuecomment-2262326073

   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your 
PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
   
   Since 2024-03-18, the Document has been moved to 
[doris-website](https://github.com/apache/doris-website).
   See [Doris 
Document](https://cwiki.apache.org/confluence/display/DORIS/Doris+Document).


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



Re: [PR] [draft](cloud-merge) Support shadow tablet to do cumulative compaction in cloud mode [doris]

2024-08-01 Thread via GitHub


Lchangliang commented on PR #37293:
URL: https://github.com/apache/doris/pull/37293#issuecomment-2262326100

   run buildall


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



Re: [PR] [pipeline](fix) Fix blocking task which is not triggered by 2nd RPC (… [doris]

2024-08-01 Thread via GitHub


Gabriel39 commented on PR #38694:
URL: https://github.com/apache/doris/pull/38694#issuecomment-2262326175

   run buildall


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



Re: [PR] [feature](http action)Add http action to show nested inverted index file (#38272) [doris]

2024-08-01 Thread via GitHub


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


##
be/src/olap/rowset/beta_rowset.cpp:
##
@@ -702,4 +703,131 @@ Status BetaRowset::calc_local_file_crc(uint32_t* 
crc_value, int64_t* file_count)
 return Status::OK();
 }
 
+Status BetaRowset::show_nested_index_file(rapidjson::Value* rowset_value,

Review Comment:
   warning: function 'show_nested_index_file' exceeds recommended 
size/complexity thresholds [readability-function-size]
   ```cpp
   Status BetaRowset::show_nested_index_file(rapidjson::Value* rowset_value,
  ^
   ```
   
   Additional context
   
   **be/src/olap/rowset/beta_rowset.cpp:705:** 124 lines including whitespace 
and comments (threshold 80)
   ```cpp
   Status BetaRowset::show_nested_index_file(rapidjson::Value* rowset_value,
  ^
   ```
   
   
   



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



[PR] [pipeline](fix) Fix blocking task which is not triggered by 2nd RPC (… [doris]

2024-08-01 Thread via GitHub


Gabriel39 opened a new pull request, #38694:
URL: https://github.com/apache/doris/pull/38694

   …#38568)
   
   Once a query is cancelled due to any reason, BE may not receive 2nd RPC from 
FE. If so, we must ensure the execution dependency is ready so tasks will not 
be blocked.
   
   ## Proposed changes
   
   Issue Number: close #xxx
   
   
   
   


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



Re: [PR] [fix](compatibility) type toSql should return lowercase string (#38012) [doris]

2024-08-01 Thread via GitHub


morrySnow commented on PR #38521:
URL: https://github.com/apache/doris/pull/38521#issuecomment-2262327894

   run buildall


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



Re: [PR] [fix](Nereids) remove db readlock before get table from db [doris]

2024-08-01 Thread via GitHub


morrySnow commented on PR #38660:
URL: https://github.com/apache/doris/pull/38660#issuecomment-2262329132

   run cloud_p0


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



Re: [PR] [fix](local exchange) Fix local exchange blocked by a huge data block… [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38693:
URL: https://github.com/apache/doris/pull/38693#issuecomment-2262330631

   clang-tidy review says "All clean, LGTM! :+1:"


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



Re: [PR] [feature](cloud) Add lazy commit mechanism for `commit_txn` [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38243:
URL: https://github.com/apache/doris/pull/38243#issuecomment-2262331605

   
   
   ClickBench: Total hot run time: 30.72 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit f40d3af909e3a4ce446ee5217827a6811bce34d1, 
data reload: false
   
   query1   0.040.030.03
   query2   0.070.030.04
   query3   0.230.050.04
   query4   1.710.070.09
   query5   0.490.490.50
   query6   1.140.720.72
   query7   0.020.020.02
   query8   0.050.040.04
   query9   0.560.520.51
   query10  0.570.570.55
   query11  0.160.120.12
   query12  0.150.120.12
   query13  0.610.600.60
   query14  0.780.790.77
   query15  0.900.860.87
   query16  0.350.360.35
   query17  1.010.980.98
   query18  0.220.200.20
   query19  1.871.751.73
   query20  0.010.010.01
   query21  15.40   0.780.67
   query22  3.846.742.14
   query23  18.03   1.281.21
   query24  2.240.230.23
   query25  0.180.080.07
   query26  0.320.210.21
   query27  0.460.240.23
   query28  13.16   1.010.97
   query29  12.53   3.323.29
   query30  0.260.050.06
   query31  2.850.410.40
   query32  3.240.490.50
   query33  2.952.943.00
   query34  15.44   4.264.29
   query35  4.294.304.26
   query36  0.670.470.48
   query37  0.190.170.16
   query38  0.160.140.15
   query39  0.040.040.03
   query40  0.160.130.13
   query41  0.100.050.05
   query42  0.060.040.05
   query43  0.040.050.04
   Total cold run time: 107.55 s
   Total hot run time: 30.72 s
   ```
   
   


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



[PR] [fix](clone) fix stale tablet report miss the new cloning replica [doris]

2024-08-01 Thread via GitHub


yujun777 opened a new pull request, #38695:
URL: https://github.com/apache/doris/pull/38695

   BUG:
   1. BE begin collect tablet report;
   2. BE clone a new replica A;
   3. FE handle this BE's tablet report from step 1. But it's stale, it don't 
include the replica A,  then FE mark replica A as bad;
   
   only after 1min later, BE report tablets again, then the new report contains 
replica A, only after that, FE will change replica A from bad to good.
   
   Fix:
   If BE clone a new replica,   it should increase its report version and tell 
FE to update it.   Then if FE handle the stale tablet report,  it will compare 
BE's report version, then found the tablet report is stale and discard it.
   
   


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



Re: [PR] [fix](clone) fix stale tablet report miss the new cloning replica [doris]

2024-08-01 Thread via GitHub


doris-robot commented on PR #38695:
URL: https://github.com/apache/doris/pull/38695#issuecomment-2262333990

   Thank you for your contribution to Apache Doris.
   Don't know what should be done next? See [How to process your 
PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR)
   
   Since 2024-03-18, the Document has been moved to 
[doris-website](https://github.com/apache/doris-website).
   See [Doris 
Document](https://cwiki.apache.org/confluence/display/DORIS/Doris+Document).


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



Re: [PR] [fix](clone) fix stale tablet report miss the new cloning replica [doris]

2024-08-01 Thread via GitHub


yujun777 commented on PR #38695:
URL: https://github.com/apache/doris/pull/38695#issuecomment-2262334117

   run buildall


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



Re: [PR] [COMPILE](macos) do not patch bitshuffle on macos [doris]

2024-08-01 Thread via GitHub


github-actions[bot] commented on PR #38634:
URL: https://github.com/apache/doris/pull/38634#issuecomment-2262334810

   clang-tidy review says "All clean, LGTM! :+1:"


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



Re: [PR] [draft](cloud-merge) Support shadow tablet to do cumulative compaction in cloud mode [doris]

2024-08-01 Thread via GitHub


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


##
cloud/test/meta_service_job_test.cpp:
##
@@ -2294,4 +2310,212 @@ TEST(MetaServiceJobTest, SchemaChangeJobPersistTest) {
 ASSERT_EQ(res.status().code(), MetaServiceCode::OK);
 }
 
+TEST(MetaServiceJobTest, DoCompactionWhenSC) {

Review Comment:
   warning: function 'TEST' exceeds recommended size/complexity thresholds 
[readability-function-size]
   ```cpp
   TEST(MetaServiceJobTest, DoCompactionWhenSC) {
   ^
   ```
   
   Additional context
   
   **cloud/test/meta_service_job_test.cpp:2312:** 85 lines including whitespace 
and comments (threshold 80)
   ```cpp
   TEST(MetaServiceJobTest, DoCompactionWhenSC) {
   ^
   ```
   
   
   



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



  1   2   3   4   5   6   7   8   9   10   >