lidavidm commented on a change in pull request #11624:
URL: https://github.com/apache/arrow/pull/11624#discussion_r744994425
##########
File path: python/pyarrow/tests/test_table.py
##########
@@ -1746,3 +1746,29 @@ def test_table_select():
result = table.select(['f2'])
expected = pa.table([a2], ['f2'])
assert result.equals(expected)
+
+
+def test_table_group_by():
+ table = pa.table([
+ pa.array([1, 2, 3, 4, 5]),
+ pa.array(["a", "a", "b", "b", "c"]),
+ ], names=["values", "keys"])
+
+ r = table.group_by("keys", ["values"], "hash_sum")
Review comment:
Hmm, we do document the "hash_" prefix in the API docs (though I suppose
those are the C++ docs anyways). But I would prefer to hide it since it's
redundant. (How will we do the documentation on the Python side though?)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
