-
Notifications
You must be signed in to change notification settings - Fork 612
Extend pivot operator support #1238
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
Conversation
Pull Request Test Coverage Report for Build 8897486318Details
💛 - Coveralls |
#[derive(Debug, Clone, PartialEq, PartialOrd, Eq, Ord, Hash)] | ||
#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] | ||
#[cfg_attr(feature = "visitor", derive(Visit, VisitMut))] | ||
pub struct ExprWithAlias { |
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.
+1 for this type - this could replace Measure
from MATCH_RECOGNIZE
and probably lots of other similar one-off types
looks good to me! cc @alamb |
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.
Extends the parser with support for: - Multiple aggregate function calls - Aliased aggregation function calls - Aliased pivot values - also allowing arbitrary expressions as values https://cloud.google.com/bigquery/docs/reference/standard-sql/query-syntax#pivot_operator
46bf2d7
to
8dfe4f5
Compare
Thanks again |
Extends the parser with support for:
https://cloud.google.com/bigquery/docs/reference/standard-sql/query-syntax#pivot_operator