[GitHub] chaoyli commented on a change in pull request #529: Remove segment group from delta writer

chaoyli commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247429548 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the Apach

[GitHub] chaoyli commented on a change in pull request #529: Remove segment group from delta writer

chaoyli commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247429476 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the Apach

[GitHub] chaoyli commented on a change in pull request #529: Remove segment group from delta writer

chaoyli commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247427432 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the Apach

[GitHub] chaoyli commented on a change in pull request #529: Remove segment group from delta writer

chaoyli commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247429441 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the Apach

[GitHub] chaoyli commented on a change in pull request #529: Remove segment group from delta writer

chaoyli commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247429587 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the Apach

[GitHub] chaoyli commented on a change in pull request #529: Remove segment group from delta writer

chaoyli commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247429028 ## File path: be/src/olap/memtable.h ## @@ -23,6 +23,7 @@ #include "olap/schema.h" #include "olap/skip

[GitHub] chaoyli commented on a change in pull request #529: Remove segment group from delta writer

chaoyli commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247422036 ## File path: be/src/olap/delta_writer.cpp ## @@ -29,34 +31,27 @@ OLAPStatus DeltaWriter::open(WriteRequ

[GitHub] chaoyli commented on a change in pull request #529: Remove segment group from delta writer

chaoyli commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247432671 ## File path: be/src/olap/rowset/alpha_rowset_builder.cpp ## @@ -72,27 +83,47 @@ OLAPStatus AlphaRowsetB

[GitHub] kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer

kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247433683 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the

[GitHub] kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer

kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247433639 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the

[GitHub] kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer

kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247433574 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the

[GitHub] chenhao7253886 opened a new pull request #531: Add cpu and io indicates to audit log

chenhao7253886 opened a new pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531 Record query consumption into fe audit log. Its basic mode of work is as follows, one of instance of parent plan is responsible for accumulating sub

[GitHub] chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log

chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247478037 ## File path: fe/src/main/java/org/apache/doris/planner/Planner.java ## @@ -230,4 +234,38 @@ priva

[GitHub] imay commented on a change in pull request #531: Add cpu and io indicates to audit log

imay commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247476827 ## File path: gensrc/proto/data.proto ## @@ -19,6 +19,11 @@ syntax="proto2"; package doris; +message PQ

[GitHub] imay commented on a change in pull request #531: Add cpu and io indicates to audit log

imay commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247476987 ## File path: fe/src/main/java/org/apache/doris/rpc/PQueryStatistic.java ## @@ -0,0 +1,29 @@ +// Licensed to

[GitHub] imay commented on a change in pull request #531: Add cpu and io indicates to audit log

imay commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247476801 ## File path: gensrc/proto/data.proto ## @@ -19,6 +19,11 @@ syntax="proto2"; package doris; +message PQ

[GitHub] imay commented on a change in pull request #531: Add cpu and io indicates to audit log

imay commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247477024 ## File path: fe/src/main/java/org/apache/doris/rpc/PQueryStatistic.java ## @@ -0,0 +1,29 @@ +// Licensed to

[GitHub] chaoyli opened a new pull request #532: Make reader can be compiled

chaoyli opened a new pull request #532: Make reader can be compiled URL: https://github.com/apache/incubator-doris/pull/532 This is an automated message from the Apache Git Service. To respond to the message, please log on Gi

[GitHub] imay commented on a change in pull request #531: Add cpu and io indicates to audit log

imay commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247498562 ## File path: be/src/runtime/data_stream_sender.cpp ## @@ -305,11 +318,17 @@ DataStreamSender::DataStreamSen

[GitHub] imay commented on a change in pull request #531: Add cpu and io indicates to audit log

imay commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247501678 ## File path: be/src/service/internal_service.cpp ## @@ -54,6 +54,18 @@ void PInternalServiceImpl::transmit

[GitHub] imay commented on a change in pull request #531: Add cpu and io indicates to audit log

imay commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247502290 ## File path: fe/src/main/java/org/apache/doris/qe/ConnectProcessor.java ## @@ -92,12 +93,16 @@ private void

[GitHub] morningman opened a new issue #533: QeProcessorImpl.reportExecStatus() log too much

morningman opened a new issue #533: QeProcessorImpl.reportExecStatus() log too much URL: https://github.com/apache/incubator-doris/issues/533 QeProcessorImpl.reportExecStatus() print too much log to fe.log This is an automate

[GitHub] kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer

kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247745392 ## File path: be/src/olap/memtable.h ## @@ -23,6 +23,7 @@ #include "olap/schema.h" #include "ola

[GitHub] kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer

kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247745417 ## File path: be/src/olap/rowset/rowset_builder_context.h ## @@ -0,0 +1,148 @@ +// Licensed to the

[GitHub] kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer

kangpinghuang commented on a change in pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#discussion_r247751356 ## File path: be/src/olap/rowset/alpha_rowset_builder.cpp ## @@ -72,27 +83,47 @@ OLAPStatus AlphaR

[GitHub] yiguolei opened a new pull request #534: Add storage migration

yiguolei opened a new pull request #534: Add storage migration URL: https://github.com/apache/incubator-doris/pull/534 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] kangpinghuang commented on a change in pull request #532: Make reader can be compiled

kangpinghuang commented on a change in pull request #532: Make reader can be compiled URL: https://github.com/apache/incubator-doris/pull/532#discussion_r247754146 ## File path: be/src/olap/reader.cpp ## @@ -503,43 +485,93 @@ OLAPStatus Reader::_capture_rs_readers(const R

[GitHub] morningman merged pull request #534: Add storage migration

morningman merged pull request #534: Add storage migration URL: https://github.com/apache/incubator-doris/pull/534 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, None Thi

[GitHub] chaoyli merged pull request #529: Remove segment group from delta writer

chaoyli merged pull request #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, None --

[GitHub] chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log

chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247772904 ## File path: fe/src/main/java/org/apache/doris/qe/ConnectProcessor.java ## @@ -92,12 +93,16 @@ pr

[GitHub] chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log

chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247773722 ## File path: be/src/service/internal_service.cpp ## @@ -54,6 +54,18 @@ void PInternalServiceImpl

[GitHub] chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log

chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247773893 ## File path: fe/src/main/java/org/apache/doris/rpc/PQueryStatistic.java ## @@ -0,0 +1,29 @@ +// L

[GitHub] chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log

chenhao7253886 commented on a change in pull request #531: Add cpu and io indicates to audit log URL: https://github.com/apache/incubator-doris/pull/531#discussion_r247773866 ## File path: be/src/runtime/data_stream_sender.cpp ## @@ -305,11 +318,17 @@ DataStreamSender::Dat

[GitHub] kangpinghuang commented on issue #528: DeltaWriter should not use SegmentGroup and ColumnDataWriter

kangpinghuang commented on issue #528: DeltaWriter should not use SegmentGroup and ColumnDataWriter URL: https://github.com/apache/incubator-doris/issues/528#issuecomment-454280541 Resolved by ISSUE #529 This is an automated

[GitHub] kangpinghuang commented on issue #529: Remove segment group from delta writer

kangpinghuang commented on issue #529: Remove segment group from delta writer URL: https://github.com/apache/incubator-doris/pull/529#issuecomment-454280640 Resolve ISSUE #528 This is an automated message from the Apache Git

[GitHub] kangpinghuang opened a new issue #535: PushHandler should not use SegmentGroup and ColumnDataWriter Directly

kangpinghuang opened a new issue #535: PushHandler should not use SegmentGroup and ColumnDataWriter Directly URL: https://github.com/apache/incubator-doris/issues/535 **Is your feature request related to a problem? Please describe.** PushHandler should not use SegmentGroup and ColumnData

[GitHub] kangpinghuang opened a new pull request #536: Add Rowset and RowsetBuilder for PushHandler

kangpinghuang opened a new pull request #536: Add Rowset and RowsetBuilder for PushHandler URL: https://github.com/apache/incubator-doris/pull/536 Remove SegmentGroup and ColumnDataWriter from PushHandler, replace them with Rowset and RowsetBuilder