feat: bigframes.bigquery.array_agg(SeriesGroupBy|DataFrameGroupby) #663
Conversation
product-auto-label
bot
added
size: l
chelsea-lin
force-pushed
the
main_chelsealin_arrayagg
branch
from
6e92dd6 to
c30464f
Compare
chelsea-lin
force-pushed
the
main_chelsealin_arrayagg
branch
from
f703d45 to
333d0ac
Compare
chelsea-lin
force-pushed
the
main_chelsealin_arrayagg
branch
from
1c44cef to
e2b0854
Compare
chelsea-lin
force-pushed
the
main_chelsealin_arrayagg
branch
from
824c27b to
c5cc12e
Compare
chelsea-lin
force-pushed
the
main_chelsealin_arrayagg
branch
from
c5cc12e to
3e071ad
Compare
bigframes/core/compile/compiled.py
Outdated
| bigframes.dtypes.ibis_dtype_to_bigframes_dtype(ibis_type), | ||
| ) | ||
|
|
||
| def _aggregate_helper( |
There was a problem hiding this comment.
Could we get a better name for this method, please. "helper" is very generic, so it's hard to understand what this method is doing.
There was a problem hiding this comment.
Renamed it into _aggergate_base to match the object name BaseIbisIR.
chelsea-lin
force-pushed
the
main_chelsealin_arrayagg
branch
from
a66d01b to
55c9d4f
Compare
chelsea-lin
force-pushed
the
main_chelsealin_arrayagg
branch
from
0115c73 to
ea3614e
Compare
chelsea-lin
added
the
automerge
|
Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot. |
gcf-merge-on-green
bot
removed
the
automerge
chelsea-lin
force-pushed
the
main_chelsealin_arrayagg
branch
from
ea3614e to
5596095
Compare
|
The end-to-end tests that failed are not caused by this particular change. |
This change introduces the
bigframes.bigquery.array_aggmethod forSeriesGroupByandDataFrameGroupby. By default, aggregated arrays are ordered by the underlying sorting columns. Additionally,array_aggis the inverse operation of(Series|Dataframe).explode().Fixes internal bug: 338232748🦕