You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "somandal (via GitHub)" <gi...@apache.org> on 2023/08/22 18:37:20 UTC
[GitHub] [pinot] somandal opened a new issue, #11406: Add ROWS support for aggregation window functions
somandal opened a new issue, #11406:
URL: https://github.com/apache/pinot/issues/11406
Today the aggregation window functions only support RANGE type window frames with the default frame boundaries (`UNBOUNDED PRECEDING` and `UNBOUNDED FOLLOWING`/`CURRENT ROW` depending on the existence of `ORDER BY` in the `OVER` clause). `ROWS` support exists for the `ROW_NUMBER` window function (this doesn't support RANGE type at all). We should add support for `ROWS` type frames for the aggregation window functions as well.
I'll be working on this change, please assign it to me
cc @siddharthteotia @walterddr @Jackie-Jiang
--
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-unsubscribe@pinot.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [I] [Multi Stage] Add ROWS support for aggregation window functions [pinot]
Posted by "yashmayya (via GitHub)" <gi...@apache.org>.
yashmayya commented on issue #11406:
URL: https://github.com/apache/pinot/issues/11406#issuecomment-2444302969
> We should add support for ROWS type frames for the aggregation window functions as well. For now we will do this for the default frame boundaries since sliding window support hasn't been added yet.
https://github.com/apache/pinot/pull/14273 adds this support as well as support for offset based ROWS window frames (`n PRECEDING` / `n FOLLOWING`) using a new sliding window based value aggregator mechanism. I'll update the documentation soon, there some to be multiple issues in the current documentation page for window functions.
--
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-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [I] [Multi Stage] Add ROWS support for aggregation window functions [pinot]
Posted by "Jackie-Jiang (via GitHub)" <gi...@apache.org>.
Jackie-Jiang closed issue #11406: [Multi Stage] Add ROWS support for aggregation window functions
URL: https://github.com/apache/pinot/issues/11406
--
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-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] siddharthteotia commented on issue #11406: [Multi Stage] Add ROWS support for aggregation window functions
Posted by "siddharthteotia (via GitHub)" <gi...@apache.org>.
siddharthteotia commented on issue #11406:
URL: https://github.com/apache/pinot/issues/11406#issuecomment-1688722863
@somandal will be working on 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-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [I] [Multi Stage] Add ROWS support for aggregation window functions [pinot]
Posted by "dan0nchik (via GitHub)" <gi...@apache.org>.
dan0nchik commented on issue #11406:
URL: https://github.com/apache/pinot/issues/11406#issuecomment-2120165961
Hi! Any update on this? Can I help somehow? Really need this feature!
--
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-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [I] [Multi Stage] Add ROWS support for aggregation window functions [pinot]
Posted by "Jackie-Jiang (via GitHub)" <gi...@apache.org>.
Jackie-Jiang commented on issue #11406:
URL: https://github.com/apache/pinot/issues/11406#issuecomment-2123577369
@xiangfu0 Can you confirm if this is already supported?
--
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-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org