-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[Enhancement](doris-future) Support REGR_AVGX_AVGY_COUNT aggregation functions #39278
base: master
Are you sure you want to change the base?
[Enhancement](doris-future) Support REGR_AVGX_AVGY_COUNT aggregation functions #39278
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
c8ddae3
to
333f2e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
7eacc91
to
cf6b603
Compare
run buildall |
TPC-H: Total hot run time: 37820 ms
|
TPC-DS: Total hot run time: 191622 ms
|
ClickBench: Total hot run time: 30.99 s
|
run buildall |
TPC-H: Total hot run time: 38302 ms
|
TPC-DS: Total hot run time: 191900 ms
|
ClickBench: Total hot run time: 31.42 s
|
/** | ||
* RegrAvgXToAvg | ||
*/ | ||
public class RegrAvgXToAvg implements ExpressionPatternRuleFactory { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
regr_avgx and regr_avgy is not same with avg, we could not rewrite them to avg. refer to https://docs.snowflake.com/en/sql-reference/functions/regr_avgx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
|
||
#pragma once | ||
|
||
#include <glog/logging.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: 'glog/logging.h' file not found [clang-diagnostic-error]
#include <glog/logging.h>
^
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems lost avgx and avgy?
Proposed changes
Issue Number: close #38974
doc:apache/doris-website#1042