[GitHub] [doris] yiguolei merged pull request #19762: [Improvement](memory) faststring use Allocator
yiguolei merged PR #19762: URL: https://github.com/apache/doris/pull/19762 -- 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
[GitHub] [doris] github-actions[bot] commented on pull request #19762: [Improvement](memory) faststring use Allocator
github-actions[bot] commented on PR #19762: URL: https://github.com/apache/doris/pull/19762#issuecomment-1552587148 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
[GitHub] [doris] github-actions[bot] commented on pull request #19762: [Improvement](memory) faststring use Allocator
github-actions[bot] commented on PR #19762: URL: https://github.com/apache/doris/pull/19762#issuecomment-1552587210 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
[doris] branch master updated (7c8b7878cd -> 068a32bc49)
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 7c8b7878cd [fix](memory) Print all query/load memory before memory GC when `memory_debug=true` (#19720) add 068a32bc49 [Improvement](memory) faststring use Allocator #19762 No new revisions were added by this update. Summary of changes: be/src/common/exception.h| 29 + be/src/olap/rowset/segment_v2/page_io.cpp| 4 +-- be/src/runtime/memory/chunk_allocator.cpp| 7 ++-- be/src/runtime/memory/mem_tracker_limiter.cpp| 8 ++--- be/src/runtime/memory/mem_tracker_limiter.h | 2 +- be/src/runtime/memory/system_allocator.cpp | 2 +- be/src/runtime/memory/system_allocator.h | 2 +- be/src/util/block_compression.cpp| 1 + be/src/util/faststring.cc| 10 +++--- be/src/util/faststring.h | 10 +++--- be/src/util/slice.h | 16 ++--- be/src/vec/common/allocator.cpp | 41 +--- be/src/vec/common/allocator.h| 14 +--- be/src/vec/common/allocator_fwd.h| 2 +- be/src/vec/common/hash_table/phmap_fwd_decl.h| 2 +- be/src/vec/core/block.cpp| 4 +-- be/test/runtime/memory/system_allocator_test.cpp | 4 +-- 17 files changed, 90 insertions(+), 68 deletions(-) - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
Error while running github feature from .asf.yaml in doris!
An error occurred while running github feature in .asf.yaml!: You can only have a maximum of 10 external triage collaborators, please contact vp-in...@apache.org to request an exception. - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
[GitHub] [doris] DongLiang-0 opened a new pull request, #19794: [typo](doc)flink and spark connector remove thrift compiled documents
DongLiang-0 opened a new pull request, #19794: URL: https://github.com/apache/doris/pull/19794 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] xinyiZzz opened a new pull request, #19795: [fix](memory) `vm/overcommit_memory` defaults to 1 in start_be.sh
xinyiZzz opened a new pull request, #19795: URL: https://github.com/apache/doris/pull/19795 # Proposed changes Issue Number: close #xxx ## Problem summary system will no longer throw bad_alloc, memory alloc are always accepted, memory limit check is handed over to Doris Allocator, make sure that the place where the exception is thrown is controllable, otherwise bad_alloc can be thrown anywhere and it will be difficult to achieve exception safety. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] github-actions[bot] commented on pull request #19795: [fix](memory) `vm/overcommit_memory` defaults to 1 in start_be.sh
github-actions[bot] commented on PR #19795:
URL: https://github.com/apache/doris/pull/19795#issuecomment-1552596075
`sh-checker report`
To get the full details, please check in the
[job]("https://github.com/apache/doris/actions/runs/5011451351";) output.
shellcheck errors
```
'shellcheck ' returned error 1 finding the following syntactical issues:
--
In bin/start_be.sh line 23:
if [ $(id -u) -eq 0 ]; then # current user has root privileges
^^ SC2292 (style): Prefer [[ ]] over [ ] for tests in
Bash/Ksh.
^--^ SC2046 (warning): Quote this to prevent word splitting.
Did you mean:
if [[ $(id -u) -eq 0 ]]; then # current user has root privileges
In bin/start_be.sh line 27:
sudo echo 1 > /proc/sys/vm/overcommit_memory
^-- SC2024 (warning): sudo doesn't affect redirects. Use ..|
sudo tee file
For more information:
https://www.shellcheck.net/wiki/SC2024 -- sudo doesn't affect redirects.
Us...
https://www.shellcheck.net/wiki/SC2046 -- Quote this to prevent word
splitt...
https://www.shellcheck.net/wiki/SC2292 -- Prefer [[ ]] over [ ] for tests
i...
--
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 ' returned error 1 finding the following formatting issues:
--
--- bin/start_be.sh.orig
+++ bin/start_be.sh
@@ -24,7 +24,7 @@
# system will no longer throw bad_alloc, memory alloc are always
accepted,
# memory limit check is handed over to Doris Allocator, make sure that
the place where the exception is thrown is controllable,
# otherwise bad_alloc can be thrown anywhere and it will be difficult
to achieve exception safety.
-sudo echo 1 > /proc/sys/vm/overcommit_memory
+sudo echo 1 >/proc/sys/vm/overcommit_memory
fi
MACHINE_OS=$(uname -s)
--
You can reformat the above files to meet shfmt's requirements by typing:
shfmt -w filename
```
--
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
[GitHub] [doris] github-actions[bot] commented on pull request #19795: [fix](memory) `vm/overcommit_memory` defaults to 1 in start_be.sh
github-actions[bot] commented on PR #19795:
URL: https://github.com/apache/doris/pull/19795#issuecomment-1552599286
`sh-checker report`
To get the full details, please check in the
[job]("https://github.com/apache/doris/actions/runs/5011465337";) output.
shellcheck errors
```
'shellcheck ' returned error 1 finding the following syntactical issues:
--
In bin/start_be.sh line 23:
if [ $(id -u) -eq 0 ]; then # current user has root privileges
^^ SC2292 (style): Prefer [[ ]] over [ ] for tests in
Bash/Ksh.
^--^ SC2046 (warning): Quote this to prevent word splitting.
Did you mean:
if [[ $(id -u) -eq 0 ]]; then # current user has root privileges
In bin/start_be.sh line 27:
sudo echo 1 > /proc/sys/vm/overcommit_memory
^-- SC2024 (warning): sudo doesn't affect redirects. Use ..|
sudo tee file
For more information:
https://www.shellcheck.net/wiki/SC2024 -- sudo doesn't affect redirects.
Us...
https://www.shellcheck.net/wiki/SC2046 -- Quote this to prevent word
splitt...
https://www.shellcheck.net/wiki/SC2292 -- Prefer [[ ]] over [ ] for tests
i...
--
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 ' returned error 1 finding the following formatting issues:
--
--- bin/start_be.sh.orig
+++ bin/start_be.sh
@@ -24,7 +24,7 @@
# system will no longer throw bad_alloc, memory alloc are always
accepted,
# memory limit check is handed over to Doris Allocator, make sure throw
exception position is controllable,
# otherwise bad_alloc can be thrown anywhere and it will be difficult
to achieve exception safety.
-sudo echo 1 > /proc/sys/vm/overcommit_memory
+sudo echo 1 >/proc/sys/vm/overcommit_memory
fi
MACHINE_OS=$(uname -s)
--
You can reformat the above files to meet shfmt's requirements by typing:
shfmt -w filename
```
--
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
[GitHub] [doris] zy-kkk opened a new pull request, #19796: [fix](jdbc catalog)fixed oceanbase catalog row limit bug
zy-kkk opened a new pull request, #19796: URL: https://github.com/apache/doris/pull/19796 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] yiguolei commented on a diff in pull request #19789: [feature](profile)Add the filtering information of the Bloom filter in profile.
yiguolei commented on code in PR #19789:
URL: https://github.com/apache/doris/pull/19789#discussion_r1197481595
##
be/src/olap/bloom_filter_predicate.h:
##
@@ -30,15 +30,27 @@
namespace doris {
// only use in runtime filter and segment v2
+
+class BloomFilterColumnHleper : public ColumnPredicate {
+public:
+BloomFilterColumnHleper(uint32_t column_id, int id)
Review Comment:
I think you could find a more common method to do it for all predicate not
only for bloomfilter. For example, you could add a get_id() or to_string method
to columnpredicate.
and you could save a map to save the stats for
every predicate.
So if we want to add other filter stats for a new predicate (not bloom
filter), it could be more expansible.
--
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
[GitHub] [doris] yiguolei merged pull request #19753: [Chore](build) fix some undefined behavior about incomplete type vector
yiguolei merged PR #19753: URL: https://github.com/apache/doris/pull/19753 -- 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 (068a32bc49 -> a2c9ed7be8)
This is an automated email from the ASF dual-hosted git repository. yiguolei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 068a32bc49 [Improvement](memory) faststring use Allocator #19762 add a2c9ed7be8 [Chore](build) fix some undefined behavior about incomplete type vector #19753 No new revisions were added by this update. Summary of changes: gensrc/thrift/AgentService.thrift | 12 ++-- gensrc/thrift/Descriptors.thrift| 18 +++--- thirdparty/patches/mysql-5.7.18-boost.patch | 20 --- thirdparty/patches/mysql-5.7.18.patch | 21 --- thirdparty/patches/rocksdb-5.14.2.patch | 85 + 5 files changed, 100 insertions(+), 56 deletions(-) delete mode 100644 thirdparty/patches/mysql-5.7.18-boost.patch delete mode 100644 thirdparty/patches/mysql-5.7.18.patch - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
Error while running github feature from .asf.yaml in doris!
An error occurred while running github feature in .asf.yaml!: You can only have a maximum of 10 external triage collaborators, please contact vp-in...@apache.org to request an exception. - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
[GitHub] [doris] bobhan1 commented on pull request #19793: [refactor-WIP](TaskWorkerPool) add DropTableTaskPool for DROP_TABLE task
bobhan1 commented on PR #19793: URL: https://github.com/apache/doris/pull/19793#issuecomment-1552617781 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
[GitHub] [doris] deadlinefen opened a new pull request, #19797: [test](client-cache) dcheck-test
deadlinefen opened a new pull request, #19797: URL: https://github.com/apache/doris/pull/19797 ONLY FOR TEST -- 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
[GitHub] [doris] deadlinefen closed pull request #19797: [test](client-cache) dcheck-test
deadlinefen closed pull request #19797: [test](client-cache) dcheck-test URL: https://github.com/apache/doris/pull/19797 -- 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
[GitHub] [doris] github-actions[bot] commented on pull request #19797: [test](client-cache) dcheck-test
github-actions[bot] commented on PR #19797: URL: https://github.com/apache/doris/pull/19797#issuecomment-1552631675 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
[GitHub] [doris] GoGoWen commented on a diff in pull request #19413: [Feature] (Multi-Catalog) support query hll column in doris jdbc table - part 1
GoGoWen commented on code in PR #19413:
URL: https://github.com/apache/doris/pull/19413#discussion_r1197494256
##
be/src/vec/data_types/data_type_hll.cpp:
##
@@ -126,4 +126,30 @@ void DataTypeHLL::to_string(const class
doris::vectorized::IColumn& column, size
ostr.write(result.c_str(), result.size());
}
+std::string DataTypeHLL::to_string(const IColumn& column, size_t row_num)
const {
+auto col_row = check_column_const_set_readability(column, row_num);
+ColumnPtr ptr = col_row.first;
+row_num = col_row.second;
+
+auto& data = const_cast(assert_cast(*ptr).get_element(row_num));
+
+std::string result(data.max_serialized_size(), '0');
+data.serialize((uint8_t*)result.data());
+return result;
+}
+
+Status DataTypeHLL::from_string(ReadBuffer& rb, IColumn* column) const {
+auto& data_column = assert_cast(*column);
+auto& data = data_column.get_data();
+
+std::string str;
+str = rb.to_string();
+HyperLogLog hll;
+if (!hll.deserialize(Slice(str))) {
+LOG(WARNING) << "deserialize hll from string fail!";
Review Comment:
in previous design, if some error hll row deserialize fail can not effect
other correct hll.And now seems we make it fail fast should be better.
--
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
[GitHub] [doris] GoGoWen commented on a diff in pull request #19413: [Feature] (Multi-Catalog) support query hll column in doris jdbc table - part 1
GoGoWen commented on code in PR #19413:
URL: https://github.com/apache/doris/pull/19413#discussion_r1197494870
##
be/src/vec/exec/vjdbc_connector.cpp:
##
@@ -648,6 +724,8 @@ Status JdbcConnector::_register_func_id(JNIEnv* env) {
RETURN_IF_ERROR(register_id(_executor_clazz, "getBlock",
JDBC_EXECUTOR_GET_BLOCK_SIGNATURE,
_executor_get_blocks_id));
+RETURN_IF_ERROR(register_id(_executor_clazz, "getBlock",
JDBC_EXECUTOR_GET_BLOCK_NEW_SIGNATURE,
Review Comment:
will change to JDBC_EXECUTOR_GET_BLOCK_WITH_TYPES_SIGNATURE, is that OK?
--
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
[GitHub] [doris] Kikyou1997 commented on pull request #19729: [fix](test) Comment unstable stats test
Kikyou1997 commented on PR #19729: URL: https://github.com/apache/doris/pull/19729#issuecomment-1552647761 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
[GitHub] [doris] deadlinefen opened a new pull request, #19798: [test](client-cache) Reopen dcheck
deadlinefen opened a new pull request, #19798: URL: https://github.com/apache/doris/pull/19798 ONLY FOR TEST -- 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
[GitHub] [doris] Kikyou1997 commented on pull request #19716: [refactor](statistics) refactor the related code of analysis info.
Kikyou1997 commented on PR #19716: URL: https://github.com/apache/doris/pull/19716#issuecomment-1552652022 I've told you many and many times that DO NOT mix the pipeline fix to a big PR like this -- 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
[GitHub] [doris] zy-kkk commented on pull request #19796: [fix](jdbc catalog)fixed oceanbase catalog row limit bug
zy-kkk commented on PR #19796: URL: https://github.com/apache/doris/pull/19796#issuecomment-1552653449 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
[GitHub] [doris] github-actions[bot] commented on pull request #19251: [Refactor](spark load) remove parquet scanner
github-actions[bot] commented on PR #19251: URL: https://github.com/apache/doris/pull/19251#issuecomment-1552653899 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
[GitHub] [doris] weizhengte opened a new pull request, #19799: [regression-test](p0) temporarily comment out unstable stats p0
weizhengte opened a new pull request, #19799: URL: https://github.com/apache/doris/pull/19799 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] github-actions[bot] commented on pull request #19798: [test](client-cache) Reopen dcheck
github-actions[bot] commented on PR #19798: URL: https://github.com/apache/doris/pull/19798#issuecomment-1552654767 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
[GitHub] [doris] weizhengte commented on pull request #19799: [regression-test](p0) temporarily comment out unstable stats p0
weizhengte commented on PR #19799: URL: https://github.com/apache/doris/pull/19799#issuecomment-1552654911 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
[GitHub] [doris] deadlinefen commented on pull request #19798: [test](client-cache) Reopen dcheck
deadlinefen commented on PR #19798: URL: https://github.com/apache/doris/pull/19798#issuecomment-1552656000 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
[GitHub] [doris] HappenLee commented on pull request #19783: [Bug](pipeline) RegressionTest failed release resouce cause DCHECK failed
HappenLee commented on PR #19783: URL: https://github.com/apache/doris/pull/19783#issuecomment-1552656257 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
[GitHub] [doris] zhengshiJ commented on pull request #19650: [Feature](Nereids) support advance mv
zhengshiJ commented on PR #19650: URL: https://github.com/apache/doris/pull/19650#issuecomment-1552660097 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
[GitHub] [doris] github-actions[bot] commented on pull request #19783: [Bug](pipeline) RegressionTest failed release resouce cause DCHECK failed
github-actions[bot] commented on PR #19783: URL: https://github.com/apache/doris/pull/19783#issuecomment-1552660148 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
[GitHub] [doris] zclllyybb commented on pull request #19072: [optimization](be)eliminate virtual calls when write mysql results
zclllyybb commented on PR #19072: URL: https://github.com/apache/doris/pull/19072#issuecomment-1552661843 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
[GitHub] [doris] zclllyybb commented on pull request #19630: [chore](stacktrace) Make crash information clearer
zclllyybb commented on PR #19630: URL: https://github.com/apache/doris/pull/19630#issuecomment-1552664743 run build all -- 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
[GitHub] [doris] zclllyybb commented on pull request #19630: [chore](stacktrace) Make crash information clearer
zclllyybb commented on PR #19630: URL: https://github.com/apache/doris/pull/19630#issuecomment-1552665139 run build all -- 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
[GitHub] [doris] github-actions[bot] commented on pull request #19072: [optimization](be)eliminate virtual calls when write mysql results
github-actions[bot] commented on PR #19072: URL: https://github.com/apache/doris/pull/19072#issuecomment-155214 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
[GitHub] [doris] zclllyybb commented on pull request #19633: [chore](LOG) fix ambiguous start time log
zclllyybb commented on PR #19633: URL: https://github.com/apache/doris/pull/19633#issuecomment-1552668414 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
[GitHub] [doris] zclllyybb commented on pull request #19647: [fix](fuzzy) make fuzzy config of BE really take effect
zclllyybb commented on PR #19647: URL: https://github.com/apache/doris/pull/19647#issuecomment-1552669632 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
[GitHub] [doris] github-actions[bot] commented on pull request #19630: [chore](stacktrace) Make crash information clearer
github-actions[bot] commented on PR #19630: URL: https://github.com/apache/doris/pull/19630#issuecomment-1552671266 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
[GitHub] [doris] zclllyybb commented on pull request #19669: [chore](build) Support specifying output path
zclllyybb commented on PR #19669: URL: https://github.com/apache/doris/pull/19669#issuecomment-1552671181 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
[GitHub] [doris] github-actions[bot] commented on pull request #19647: [fix](fuzzy) make fuzzy config of BE really take effect
github-actions[bot] commented on PR #19647: URL: https://github.com/apache/doris/pull/19647#issuecomment-1552676060 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
[GitHub] [doris] Kikyou1997 commented on pull request #19729: [fix](test) Comment unstable stats test
Kikyou1997 commented on PR #19729: URL: https://github.com/apache/doris/pull/19729#issuecomment-1552677960 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
[GitHub] [doris] github-actions[bot] commented on pull request #19099: [performance](stream-load) use vector instead of skiplist when insert agg keys
github-actions[bot] commented on PR #19099: URL: https://github.com/apache/doris/pull/19099#issuecomment-1552679735 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
[GitHub] [doris] adonis0147 opened a new pull request, #19800: [bug](MTMV) Fix the wrong interpretation for NEVER REFRESH
adonis0147 opened a new pull request, #19800: URL: https://github.com/apache/doris/pull/19800 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] stalary closed issue #18781: [Bug] doris-1.2.2 hive catalog 不能自动同步表信息
stalary closed issue #18781: [Bug] doris-1.2.2 hive catalog 不能自动同步表信息 URL: https://github.com/apache/doris/issues/18781 -- 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
[GitHub] [doris] adonis0147 commented on pull request #19800: [bug](MTMV) Fix the wrong interpretation for NEVER REFRESH
adonis0147 commented on PR #19800: URL: https://github.com/apache/doris/pull/19800#issuecomment-1552680138 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
[GitHub] [doris] github-actions[bot] commented on pull request #19099: [performance](stream-load) use vector instead of skiplist when insert agg keys
github-actions[bot] commented on PR #19099: URL: https://github.com/apache/doris/pull/19099#issuecomment-1552683256 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
[GitHub] [doris] yongjinhou commented on issue #19784: [Bug] Query TVF failed
yongjinhou commented on issue #19784: URL: https://github.com/apache/doris/issues/19784#issuecomment-1552683481 Bug fixing -- 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
[GitHub] [doris] Gabriel39 opened a new pull request, #19801: [Bug](DecimalV3) fix decimalv3 functions
Gabriel39 opened a new pull request, #19801: URL: https://github.com/apache/doris/pull/19801 # Proposed changes fix decimalv3 functions ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] Gabriel39 commented on pull request #19801: [Bug](DecimalV3) fix decimalv3 functions
Gabriel39 commented on PR #19801: URL: https://github.com/apache/doris/pull/19801#issuecomment-1552683766 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
[GitHub] [doris] github-actions[bot] commented on pull request #19641: [DRAFT](decimalv3) test auto conversion
github-actions[bot] commented on PR #19641: URL: https://github.com/apache/doris/pull/19641#issuecomment-1552684953 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
[GitHub] [doris] github-actions[bot] commented on pull request #19099: [performance](stream-load) use vector instead of skiplist when insert agg keys
github-actions[bot] commented on PR #19099: URL: https://github.com/apache/doris/pull/19099#issuecomment-1552685795 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
[GitHub] [doris] github-actions[bot] commented on pull request #19801: [Bug](DecimalV3) fix decimalv3 functions
github-actions[bot] commented on PR #19801: URL: https://github.com/apache/doris/pull/19801#issuecomment-1552686909 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
[GitHub] [doris] github-actions[bot] commented on pull request #19795: [fix](memory) `vm/overcommit_memory` defaults to 1 in start_be.sh
github-actions[bot] commented on PR #19795:
URL: https://github.com/apache/doris/pull/19795#issuecomment-1552687646
`sh-checker report`
To get the full details, please check in the
[job]("https://github.com/apache/doris/actions/runs/5011824541";) output.
shellcheck errors
```
'shellcheck ' returned error 1 finding the following syntactical issues:
--
In bin/start_be.sh line 27:
sudo echo 1 >/proc/sys/vm/overcommit_memory
^-- SC2024 (warning): sudo doesn't affect redirects. Use ..|
sudo tee file
For more information:
https://www.shellcheck.net/wiki/SC2024 -- sudo doesn't affect redirects.
Us...
--
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
[GitHub] [doris] github-actions[bot] commented on pull request #19262: [Feature](planner) use partial update in update from & delete from
github-actions[bot] commented on PR #19262: URL: https://github.com/apache/doris/pull/19262#issuecomment-1552688425 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
[GitHub] [doris] luozenglin opened a new pull request, #19802: [feature-wip](resouce-group) Supports memory soft isolation of resource group
luozenglin opened a new pull request, #19802: URL: https://github.com/apache/doris/pull/19802 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] englefly commented on pull request #19673: [feature](nereids) set proper min/max value for column stats when minExpr/maxExpr is not avialable
englefly commented on PR #19673: URL: https://github.com/apache/doris/pull/19673#issuecomment-1552691017 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
[GitHub] [doris] DongLiang-0 commented on pull request #19794: [typo](doc)flink and spark connector remove thrift compiled documents
DongLiang-0 commented on PR #19794: URL: https://github.com/apache/doris/pull/19794#issuecomment-1552691669 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
[GitHub] [doris] github-actions[bot] commented on pull request #19802: [feature-wip](resouce-group) Supports memory soft isolation of resource group
github-actions[bot] commented on PR #19802: URL: https://github.com/apache/doris/pull/19802#issuecomment-1552692902 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
[GitHub] [doris] englefly commented on pull request #19742: [improve](nereids) check be status when column stats is unknown
englefly commented on PR #19742: URL: https://github.com/apache/doris/pull/19742#issuecomment-1552693427 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
[GitHub] [doris] LemonLiTree opened a new pull request, #19803: [bug](date_format)fix date_format parameter contains %v result error
LemonLiTree opened a new pull request, #19803:
URL: https://github.com/apache/doris/pull/19803
# Proposed changes
Issue Number: close [#19761](https://github.com/apache/doris/issues/19761)
## Problem summary
```sql
mysql> select date_format ('2023-05-07',"%x-%v");
+-+
| date_format('2023-05-07 00:00:00', '%x-%v') |
+-+
| 2023-18 |
+-+
1 row in set (0.02 sec)
mysql> select date_format ('2022-01-01',"%x-%v");
+-+
| date_format('2022-01-01 00:00:00', '%x-%v') |
+-+
| 2021-52 |
+-+
1 row in set (0.01 sec)
```
## Checklist(Required)
* [ ] Does it affect the original behavior
* [ ] Has unit tests been added
* [ ] Has document been added or modified
* [ ] Does it need to update dependencies
* [ ] Is this PR support rollback (If NO, please explain WHY)
## Further comments
If this is a relatively large or complex change, kick off the discussion at
[d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you
chose the solution you did and what alternatives you considered, etc...
--
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
[GitHub] [doris] luozenglin commented on pull request #19802: [feature-wip](resouce-group) Supports memory soft isolation of resource group
luozenglin commented on PR #19802: URL: https://github.com/apache/doris/pull/19802#issuecomment-1552696682 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
[GitHub] [doris] airborne12 commented on pull request #19352: [Update](build) change clucene from thirdparty to git module
airborne12 commented on PR #19352: URL: https://github.com/apache/doris/pull/19352#issuecomment-1552701316 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
[GitHub] [doris] github-actions[bot] commented on pull request #19352: [Update](build) change clucene from thirdparty to git module
github-actions[bot] commented on PR #19352: URL: https://github.com/apache/doris/pull/19352#issuecomment-1552704164 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
[GitHub] [doris] englefly commented on pull request #19717: [feature](nereids) merge in predicates
englefly commented on PR #19717: URL: https://github.com/apache/doris/pull/19717#issuecomment-1552706581 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
[doris] branch master updated (a2c9ed7be8 -> 294599ee45)
This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
from a2c9ed7be8 [Chore](build) fix some undefined behavior about incomplete
type vector #19753
add 294599ee45 [feature](jsonb) rename JSONB type name and function name
to JSON (#19774)
No new revisions were added by this update.
Summary of changes:
be/src/vec/functions/function_jsonb.cpp| 77 +++-
be/test/vec/function/function_jsonb_test.cpp | 70 ++--
.../{jsonb_exists_path.md => json_exists_path.md} | 12 +-
.../sql-functions/json-functions/json_extract.md | 61 ++-
.../{jsonb_parse.md => json_parse.md} | 36 +-
.../json-functions/{jsonb_type.md => json_type.md} | 12 +-
.../sql-functions/json-functions/jsonb_extract.md | 93 -
.../sql-reference/Data-Types/{JSONB.md => JSON.md} | 138 +++
docs/sidebars.json | 19 +-
.../{jsonb_exists_path.md => json_exists_path.md} | 12 +-
.../sql-functions/json-functions/json_extract.md | 41 +-
.../{jsonb_parse.md => json_parse.md} | 34 +-
.../json-functions/{jsonb_type.md => json_type.md} | 8 +-
.../sql-functions/json-functions/jsonb_extract.md | 71
.../sql-reference/Data-Types/{JSONB.md => JSON.md} | 138 +++
.../org/apache/doris/catalog/PrimitiveType.java| 2 +-
.../java/org/apache/doris/catalog/ScalarType.java | 6 +-
fe/fe-core/src/main/cup/sql_parser.cup | 5 +
.../java/org/apache/doris/common/util/Util.java| 2 +-
.../org/apache/doris/external/jdbc/JdbcClient.java | 2 +-
fe/fe-core/src/main/jflex/sql_scanner.flex | 1 +
gensrc/script/doris_builtins_functions.py | 34 +-
.../test_jsonb.csv => json_p0/test_json.csv} | 0
.../test_json_load_and_function.out} | 0
.../test_json_load_unique_key_and_function.out}| 0
.../test_json_unique_key.csv} | 0
.../data/types/test_show_data_types.out| 2 +-
.../json_p0/test_json_load_and_function.groovy | 437 +
.../test_json_load_unique_key_and_function.groovy | 406 +++
29 files changed, 1280 insertions(+), 439 deletions(-)
rename
docs/en/docs/sql-manual/sql-functions/json-functions/{jsonb_exists_path.md =>
json_exists_path.md} (76%)
rename docs/en/docs/sql-manual/sql-functions/json-functions/{jsonb_parse.md =>
json_parse.md} (64%)
rename docs/en/docs/sql-manual/sql-functions/json-functions/{jsonb_type.md =>
json_type.md} (78%)
delete mode 100644
docs/en/docs/sql-manual/sql-functions/json-functions/jsonb_extract.md
rename docs/en/docs/sql-manual/sql-reference/Data-Types/{JSONB.md => JSON.md}
(93%)
rename
docs/zh-CN/docs/sql-manual/sql-functions/json-functions/{jsonb_exists_path.md
=> json_exists_path.md} (73%)
rename docs/zh-CN/docs/sql-manual/sql-functions/json-functions/{jsonb_parse.md
=> json_parse.md} (65%)
rename docs/zh-CN/docs/sql-manual/sql-functions/json-functions/{jsonb_type.md
=> json_type.md} (87%)
delete mode 100644
docs/zh-CN/docs/sql-manual/sql-functions/json-functions/jsonb_extract.md
rename docs/zh-CN/docs/sql-manual/sql-reference/Data-Types/{JSONB.md =>
JSON.md} (93%)
copy regression-test/data/{jsonb_p0/test_jsonb.csv => json_p0/test_json.csv}
(100%)
copy regression-test/data/{jsonb_p0/test_jsonb_load_and_function.out =>
json_p0/test_json_load_and_function.out} (100%)
copy
regression-test/data/{jsonb_p0/test_jsonb_load_unique_key_and_function.out =>
json_p0/test_json_load_unique_key_and_function.out} (100%)
copy regression-test/data/{jsonb_p0/test_jsonb_unique_key.csv =>
json_p0/test_json_unique_key.csv} (100%)
create mode 100644
regression-test/suites/json_p0/test_json_load_and_function.groovy
create mode 100644
regression-test/suites/json_p0/test_json_load_unique_key_and_function.groovy
-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
Error while running github feature from .asf.yaml in doris!
An error occurred while running github feature in .asf.yaml!: You can only have a maximum of 10 external triage collaborators, please contact vp-in...@apache.org to request an exception. - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
[GitHub] [doris] yiguolei merged pull request #19774: [feature](jsonb) rename JSONB type name and function name to JSON
yiguolei merged PR #19774: URL: https://github.com/apache/doris/pull/19774 -- 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
[GitHub] [doris] github-actions[bot] commented on pull request #19800: [bug](MTMV) Fix the wrong interpretation for NEVER REFRESH
github-actions[bot] commented on PR #19800: URL: https://github.com/apache/doris/pull/19800#issuecomment-1552709054 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
[GitHub] [doris] github-actions[bot] commented on pull request #19800: [bug](MTMV) Fix the wrong interpretation for NEVER REFRESH
github-actions[bot] commented on PR #19800: URL: https://github.com/apache/doris/pull/19800#issuecomment-1552709097 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
[GitHub] [doris] englefly commented on pull request #19673: [feature](nereids) set proper min/max value for column stats when minExpr/maxExpr is not avialable
englefly commented on PR #19673: URL: https://github.com/apache/doris/pull/19673#issuecomment-1552709774 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
[GitHub] [doris] luozenglin opened a new pull request, #19804: [fix](tracing) fix the issue that a trace may track multiple queries
luozenglin opened a new pull request, #19804: URL: https://github.com/apache/doris/pull/19804 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] qidaye merged pull request #19691: [Optimize](row store) optimize serialization and deserialization
qidaye merged PR #19691: URL: https://github.com/apache/doris/pull/19691 -- 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 (294599ee45 -> fd4fa5c64e)
This is an automated email from the ASF dual-hosted git repository. jianliangqi pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from 294599ee45 [feature](jsonb) rename JSONB type name and function name to JSON (#19774) add fd4fa5c64e [Optimize](row store) optimize serialization and deserialization (#19691) No new revisions were added by this update. Summary of changes: be/src/exec/rowid_fetcher.cpp| 14 - be/src/olap/memtable.cpp | 5 +- be/src/olap/rowset/segment_v2/segment_writer.cpp | 4 +- be/src/olap/tablet.cpp | 13 - be/src/olap/tablet_schema.h | 2 +- be/src/service/point_query_executor.cpp | 18 -- be/src/service/point_query_executor.h| 10 be/src/vec/data_types/serde/data_type_serde.cpp | 21 +++ be/src/vec/data_types/serde/data_type_serde.h| 6 ++ be/src/vec/jsonb/serialize.cpp | 72 ++-- be/src/vec/jsonb/serialize.h | 18 +++--- be/test/vec/jsonb/serialize_test.cpp | 29 -- 12 files changed, 145 insertions(+), 67 deletions(-) - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
Error while running github feature from .asf.yaml in doris!
An error occurred while running github feature in .asf.yaml!: You can only have a maximum of 10 external triage collaborators, please contact vp-in...@apache.org to request an exception. - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
[GitHub] [doris] englefly commented on pull request #19640: [fix](nereids) avoid 0 row count in stats derive
englefly commented on PR #19640: URL: https://github.com/apache/doris/pull/19640#issuecomment-1552713751 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
[GitHub] [doris] yiguolei commented on pull request #19251: [Refactor](spark load) remove parquet scanner
yiguolei commented on PR #19251: URL: https://github.com/apache/doris/pull/19251#issuecomment-1552713765 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
[GitHub] [doris] adonis0147 opened a new issue, #19805: [Bug] NPE when creating a materialized view with multiple tables which never refreshes
adonis0147 opened a new issue, #19805: URL: https://github.com/apache/doris/issues/19805 ### Search before asking - [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues. ### Version master ### What's Wrong? NPE raised. ```shell ``` ### What You Expected? Fix the exception. ### How to Reproduce? ``` CREATE TABLE user1 ( k1 TINYINT, k3 INT ) COMMENT "my first table" DISTRIBUTED BY HASH(k1) BUCKETS 6 PROPERTIES ( "replication_num" = "1" ); CREATE TABLE user2 ( k1 TINYINT, k4 INT ) COMMENT "my first table" DISTRIBUTED BY HASH(k1) BUCKETS 6 PROPERTIES ( "replication_num" = "1" ); insert into user1 values(1,1); insert into user1 values(2,2); insert into user2 values(1,3); insert into user2 values(2,4); CREATE MATERIALIZED VIEW mv1 BUILD IMMEDIATE NEVER REFRESH DISTRIBUTED BY HASH(k1) BUCKETS 6 PROPERTIES ( "replication_num" = "1" ) AS SELECT user1.k1,user1.k3,user2.k4 FROM user1 JOIN user2 ON user1.k1=user2.k1; ``` ### Anything Else? _No response_ ### Are you willing to submit PR? - [X] Yes I am willing to submit a PR! ### Code of Conduct - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct) -- 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.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
[GitHub] [doris] englefly commented on pull request #19717: [feature](nereids) merge in predicates
englefly commented on PR #19717: URL: https://github.com/apache/doris/pull/19717#issuecomment-1552716880 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
[GitHub] [doris] Gabriel39 commented on pull request #19641: [DRAFT](decimalv3) test auto conversion
Gabriel39 commented on PR #19641: URL: https://github.com/apache/doris/pull/19641#issuecomment-1552718132 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
[GitHub] [doris] Gabriel39 opened a new pull request, #19806: [decimalv3](function) support function width_bucket
Gabriel39 opened a new pull request, #19806: URL: https://github.com/apache/doris/pull/19806 # Proposed changes Issue Number: close #xxx ## Problem summary Describe your changes. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] github-actions[bot] commented on pull request #19806: [decimalv3](function) support function width_bucket
github-actions[bot] commented on PR #19806: URL: https://github.com/apache/doris/pull/19806#issuecomment-1552723060 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
[GitHub] [doris] ZhangYu0123 commented on pull request #19622: [enhence](memory) gc inverted index cache when there is not enough memory
ZhangYu0123 commented on PR #19622: URL: https://github.com/apache/doris/pull/19622#issuecomment-1552725763 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
[GitHub] [doris] Level1Accelerator commented on issue #8446: 复制现有doris环境整个目录，在新环境启动导致原环境的be服务宕机
Level1Accelerator commented on issue #8446: URL: https://github.com/apache/doris/issues/8446#issuecomment-1552727543 +1,先升级fe，再升级be的过程会碰到这个问题 -- 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
[GitHub] [doris] github-actions[bot] commented on pull request #19622: [enhence](memory) gc inverted index cache when there is not enough memory
github-actions[bot] commented on PR #19622: URL: https://github.com/apache/doris/pull/19622#issuecomment-1552733056 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
[GitHub] [doris] xinyiZzz merged pull request #19622: [enhence](memory) gc inverted index cache when there is not enough memory
xinyiZzz merged PR #19622: URL: https://github.com/apache/doris/pull/19622 -- 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
[GitHub] [doris] yixiutt commented on pull request #19262: [Feature](planner) use partial update in update from & delete from
yixiutt commented on PR #19262: URL: https://github.com/apache/doris/pull/19262#issuecomment-1552733868 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
[doris] branch master updated (fd4fa5c64e -> 07bbf741fb)
This is an automated email from the ASF dual-hosted git repository. zouxinyi pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/doris.git from fd4fa5c64e [Optimize](row store) optimize serialization and deserialization (#19691) add 07bbf741fb [enhence](memory) gc inverted index cache when there is not enough memory (#19622) No new revisions were added by this update. Summary of changes: be/src/common/config.cpp | 2 + be/src/common/config.h | 2 + be/src/olap/comparison_predicate.h | 2 +- be/src/olap/in_list_predicate.h| 2 +- be/src/olap/match_predicate.cpp| 2 +- be/src/olap/null_predicate.cpp | 2 +- .../rowset/segment_v2/inverted_index_cache.cpp | 82 -- .../olap/rowset/segment_v2/inverted_index_cache.h | 22 -- .../rowset/segment_v2/inverted_index_reader.cpp| 9 +-- be/src/util/mem_info.cpp | 10 +++ 10 files changed, 119 insertions(+), 16 deletions(-) - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
Error while running github feature from .asf.yaml in doris!
An error occurred while running github feature in .asf.yaml!: You can only have a maximum of 10 external triage collaborators, please contact vp-in...@apache.org to request an exception. - To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
[GitHub] [doris] github-actions[bot] commented on pull request #19413: [Feature] (Multi-Catalog) support query hll column in doris jdbc table - part 1
github-actions[bot] commented on PR #19413: URL: https://github.com/apache/doris/pull/19413#issuecomment-1552733754 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
[GitHub] [doris] LiBinfeng-01 opened a new pull request, #19807: [Fix](Nereids) Fix bitmap type in compare operator bug
LiBinfeng-01 opened a new pull request, #19807: URL: https://github.com/apache/doris/pull/19807 # Proposed changes ## Problem summary When using nereids, if we use compare operator of bitmap type, an analyze exception need to be throwed. like: select id from (select BITMAP_EMPTY() as c0 from expr_test) as ref0 where c0 = 1 order by id Which c0 in subq0 is a bitmap type, this scenario is not supported right now. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] yongjinhou opened a new pull request, #19808: [Fix](tvf) Fix select resource groups bug
yongjinhou opened a new pull request, #19808: URL: https://github.com/apache/doris/pull/19808 # Proposed changes Issue Number: close #19784 ## Problem summary fix select resource groups bug. ## Checklist(Required) * [x] Does it affect the original behavior * [ ] Has unit tests been added * [x] Has document been added or modified * [ ] Does it need to update dependencies * [x] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] zy-kkk opened a new pull request, #19809: [improvement](jdbc catalog) Add adaptation to Oracle special character `/` table names
zy-kkk opened a new pull request, #19809: URL: https://github.com/apache/doris/pull/19809 # Proposed changes Issue Number: close #xxx ## Problem summary 在Oracle数据库中，我们可以创建带有 `/` 字符的表名，正常情况下我们需要加双引号来适配，但是在java的Metadata database中，加双引号是无效的，所以这里我做了模糊匹配，经过测试是可以的 ```sql mysql> show tables; +--+ | Tables_in_DORIS_TEST | +--+ | /AFS/MARM| | STUDENT | | TEST_CHAR| | TEST_DATE| | TEST_INSERT | | TEST_INT | | TEST_NUM | | TEST_NUMBER | | TEST_NUMBER2 | | TEST_NUMBER3 | | TEST_NUMBER4 | | TEST_RAW | | TEST_TIMESTAMP | +--+ 13 rows in set (0.01 sec) mysql> select * from `/AFS/MARM`; Empty set (0.36 sec) mysql> desc `/AFS/MARM`; +--+--+--+--+-+---+ | Field| Type | Null | Key | Default | Extra | +--+--+--+--+-+---+ | MANDT| TEXT | No | true | NULL| | | MATNR| TEXT | No | true | NULL| | | MEINH| TEXT | No | true | NULL| | | J_3ASIZE | TEXT | No | true | NULL| | | J_4KSCAT | TEXT | No | true | NULL| | | VOLUM| DECIMALV3(13, 3) | No | true | NULL| | | BRGEW| DECIMALV3(13, 3) | No | true | NULL| | | NTGEW| DECIMALV3(13, 3) | No | true | NULL| | | REFDIM | TEXT | No | true | NULL| | | ZZYYBM | TEXT | No | true | NULL| | | ZZBDBM | TEXT | No | true | NULL| | | ZZSSRQ | TEXT | No | true | NULL| | | YYLZD1 | TEXT | No | true | NULL| | | YYLZD2 | TEXT | No | true | NULL| | | YYLZD3 | TEXT | No | true | NULL| | | YYLZD4 | TEXT | No | true | NULL| | | YYLZD5 | TEXT | No | true | NULL| | | ZZXZBM | TEXT | No | true | NULL| | +--+--+--+--+-+---+ 18 rows in set (0.04 sec) ``` ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [ ] Is this PR support rollback (If NO, please explain WHY) ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mailto:d...@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc... -- 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
[GitHub] [doris] yongjinhou commented on pull request #19808: [Fix](tvf) Fix select resource groups bug
yongjinhou commented on PR #19808: URL: https://github.com/apache/doris/pull/19808#issuecomment-1552741840 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
[GitHub] [doris-website] luzhijing opened a new pull request, #231: update 404 link of 'more' in homepage
luzhijing opened a new pull request, #231: URL: https://github.com/apache/doris-website/pull/231 (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
[GitHub] [doris] zy-kkk commented on pull request #19809: [improvement](jdbc catalog) Add adaptation to Oracle special character `/` table names
zy-kkk commented on PR #19809: URL: https://github.com/apache/doris/pull/19809#issuecomment-1552745962 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
[GitHub] [doris] nextdreamblue commented on pull request #19326: [feature-wip](duplicate-no-keys) schame change support for duplicate no keys
nextdreamblue commented on PR #19326: URL: https://github.com/apache/doris/pull/19326#issuecomment-1552746500 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
[GitHub] [doris] luozenglin commented on pull request #19804: [fix](tracing) fix the issue that a trace may track multiple queries
luozenglin commented on PR #19804: URL: https://github.com/apache/doris/pull/19804#issuecomment-1552751221 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
[GitHub] [doris] github-actions[bot] commented on pull request #19795: [fix](memory) Remind log if `vm/overcommit_memory`=2 when be start
github-actions[bot] commented on PR #19795: URL: https://github.com/apache/doris/pull/19795#issuecomment-1552752898 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
[GitHub] [doris] GoGoWen commented on a diff in pull request #19413: [Feature] (Multi-Catalog) support query hll column in doris jdbc table - part 1
GoGoWen commented on code in PR #19413:
URL: https://github.com/apache/doris/pull/19413#discussion_r1197579738
##
regression-test/suites/jdbc_catalog_p0/test_doris_jdbc_catalog.groovy:
##
@@ -0,0 +1,174 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+suite("test_doris_jdbc_catalog", "p0") {
+qt_sql """select current_catalog()"""
+
+String jdbcUrl = context.config.jdbcUrl +
"&return_object_data_as_binary=true"
+String jdbcUser = context.config.jdbcUser
+String jdbcPassword = context.config.jdbcPassword
+String driver_url =
"https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/jdbc_driver/mysql-connector-java-8.0.25.jar";
+
+String resource_name = "jdbc_resource_catalog_doris"
+String catalog_name = "doris_jdbc_catalog";
+String internal_db_name = "regression_test_jdbc_catalog_p0";
+String doris_port = 9030;
+String inDorisTable = "doris_in_tb";
+String hllTable = "bowen_hll_test"
+
+qt_sql """select current_catalog()"""
+sql """drop catalog if exists ${catalog_name} """
+
+sql """ CREATE CATALOG `${catalog_name}` PROPERTIES (
+"user" = "${jdbcUser}",
+"type" = "jdbc",
+"password" = "${jdbcPassword}",
+"jdbc_url" = "${jdbcUrl}",
+"driver_url" = "${driver_url}",
+"driver_class" = "com.mysql.jdbc.Driver"
+)"""
+
+sql """ drop table if exists ${inDorisTable} """
+sql """
+ CREATE TABLE ${inDorisTable} (
+`id` INT NULL COMMENT "主键id",
+`name` string NULL COMMENT "名字"
+) DISTRIBUTED BY HASH(id) BUCKETS 10
+PROPERTIES("replication_num" = "1");
+"""
+sql """ insert into ${inDorisTable} values (1, 'doris1')"""
+sql """ insert into ${inDorisTable} values (2, 'doris2')"""
+sql """ insert into ${inDorisTable} values (3, 'doris3')"""
+sql """ insert into ${inDorisTable} values (4, 'doris4')"""
+sql """ insert into ${inDorisTable} values (5, 'doris5')"""
+sql """ insert into ${inDorisTable} values (6, 'doris6')"""
+
+order_qt_ex_tb1 """ select * from
internal.${internal_db_name}.${inDorisTable} order by id; """
+
+qt_sql """select current_catalog()"""
+sql "switch ${catalog_name}"
+qt_sql """select current_catalog()"""
+sql """ use ${internal_db_name}"""
+order_qt_ex_tb1 """ select * from ${inDorisTable} order by id; """
+
+// test hll query
+sql "switch internal"
+sql "use ${internal_db_name}"
+
+sql """ drop table if exists ${hllTable} """
+sql """ CREATE TABLE `${hllTable}` (
+ `pin_id` bigint(20) NOT NULL COMMENT "",
+ `pv_date` datev2 NOT NULL COMMENT "",
+ `user_log_acct` hll HLL_UNION NULL COMMENT ""
+) ENGINE=OLAP
+AGGREGATE KEY(`pin_id`, `pv_date`)
+COMMENT "OLAP"
+PARTITION BY RANGE(`pv_date`)
+(PARTITION pbefore201910 VALUES [('1900-01-01'), ('2019-10-01')),
+PARTITION p201910 VALUES [('2019-10-01'), ('2019-11-01')),
+PARTITION p201911 VALUES [('2019-11-01'), ('2019-12-01')),
+PARTITION p201912 VALUES [('2019-12-01'), ('2020-01-01')),
+PARTITION p202001 VALUES [('2020-01-01'), ('2020-02-01')),
+PARTITION p202002 VALUES [('2020-02-01'), ('2020-03-01')),
+PARTITION p202003 VALUES [('2020-03-01'), ('2020-04-01')),
+PARTITION p202004 VALUES [('2020-04-01'), ('2020-05-01')),
+PARTITION p202005 VALUES [('2020-05-01'), ('2020-06-01')),
+PARTITION p202006 VALUES [('2020-06-01'), ('2020-07-01')),
+PARTITION p202007 VALUES [('2020-07-01'), ('2020-08-01')),
+PARTITION p202008 VALUES [('2020-08-01'), ('2020-09-01')),
+PARTITION p202009 VALUES [('2020-09-01'), ('2020-10-01')),
+PARTITION p202010 VALUES [('2020-10-01'), ('2020-11-01')),
+PARTITION p202011 VALUES [('2020-11-01'), ('2020-12-01')),
+PARTITION p202012 VALUES [('2020-12-01'), ('2021-01-01')),
+PARTITION p202101 VALUES [('2021-01-01'), ('2021-02-01')),
+PARTITION p202102 VALUES [('2021-02-01'), ('2021-03-01')),
+PARTITION p202103 VALUES [('2021-03-01'), ('2021-04-01')),
+PARTITION p202104 VALUES [('2021
