amol- commented on a change in pull request #11624:
URL: https://github.com/apache/arrow/pull/11624#discussion_r753011605
##########
File path: python/pyarrow/table.pxi
##########
@@ -2387,3 +2429,60 @@ def _from_pydict(cls, mapping, schema, metadata):
return cls.from_arrays(arrays, schema=schema, metadata=metadata)
else:
raise TypeError('Schema must be an instance of pyarrow.Schema')
+
+
+class TableGroupBy:
+ """
+ A grouping of columns in a table on which to perform aggregations.
+ """
+
+ def __init__(self, table, keys):
+ if isinstance(keys, str):
+ keys = [keys]
+
+ self._table = table
+ self.keys = keys
+
+ def aggregate(self, aggregations):
+ """
+ Perform an aggregation over the grouped columns of the table.
+
+ Parameters
+ ----------
+ aggregations : list[tuple(str, str)] or\
+ list[tuple(str, str, FunctionOptions)]
+ List of tuples made of aggregation functions names followed
+ by column names and optionally aggregation function options.
Review comment:
Well, in pandas it's reversed because the aggregations are keyed by the
column `"col": ["sum", "count"]`, here the aggregations are just listed so
there is no benefit in having the column first and I felt it was more "natural"
if you read "sum X" our "count X" than "X count" etc...
Anyway I don't have a strong opinion so if you feel it's better to reverse
them I'm ok with that
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
