-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
In some condition use sub-query make doris return wrong value #1097
Comments
|
(2) wrong result:
|
It seems that the problem is located in our order by implementation. We will look into it and try to reproduce this problem. And if you are convenient, you can help us to simplify this query and to reproduce this problem. |
I'm very glad to help location this problem, I will try to simplify this query. ☺ |
Did you have upgraded to our lastest version? This seems to be solved before. |
The version I tried is 0.9.0, I will try the newest master branch. And it will be nice if you paste me the patch commit log~ |
OK, i will conform whether it has been fixed. |
The special thing is that I use count(distinct cuid) instead of HLL_UNION_AGG(cuid_uv_set), after that the result is very weird. |
Seems when I use count distinct as the denominator will lead to wrong result. |
When I use (cast(sum(read_duration) as bigint))/count(distinct cuid), the result become correct. I guess The function is overflowed |
what is column |
read_duration : DECIMAL(20,2) |
OK, it seems there is a bug in DECIMAL, thank you, I will track it ASAP. |
Can you add my wechat "leanderleereed"? I want to talk more detail about this issue? |
Sure~ |
This bug has been fixed in the tag of Doris 0.9.0, and please upgrade your Doris. |
Seems version 0.9.0-rc02 has solved this bug, Thx! |
…o fix memtracker (apache#1172) * [Fix](memory-leak) Fix boost::stacktrace memory leak (apache#1097) * [Fix](selectdb-cloud) Several picks to fix memtracker (apache#1087) * [enhancement](memtracker) Add independent and unique scanner mem tracker for each query (apache#13262) * [enhancement](memory) Print memory usage log when memory allocation fails (apache#13301) * [enhancement](memtracker) Print query memory usage log every second when `memory_verbose_track` is enabled (apache#13302) * [fix](memory) Fix USE_JEMALLOC=true UBSAN compilation error apache#13398 * [enhancement](memtracker) Fix brpc causing query mem tracker to be inaccurate apache#13401 * [fix](memtracker) Fix transmit_tracker null pointer because phamp is not thread safe apache#13528 * [enhancement](memtracker) Fix Brpc mem count and refactored thread context macro (apache#13469) * [enhancement](memtracker) Refactor load channel + memtable mem tracker (apache#13795) * [fix](load) Fix load channel mgr lock (apache#13960) hot fix load channel mgr lock * [fix](memtracker) Fix DCHECK !std::count(_consumer_tracker_stack.begin(), _consumer_tracker_stack.end(), tracker) * [tempfix][memtracker] wait pick 0b945fe Co-authored-by: Xinyi Zou <zouxinyi02@gmail.com>
Describe the bug
To Reproduce
Steps to reproduce the behavior:
The text was updated successfully, but these errors were encountered: