You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/04/12 09:27:42 UTC

[GitHub] [arrow] AlenkaF opened a new pull request, #12863: ARROW-11259: [Python] Allow to create field reference to nested field

AlenkaF opened a new pull request, #12863:
URL: https://github.com/apache/arrow/pull/12863

   This PR tries to redo the work from https://github.com/apache/arrow/pull/9799.
   
   It will unblock:
   - https://issues.apache.org/jira/browse/ARROW-13798
   - https://issues.apache.org/jira/browse/ARROW-14596
   
   cc @jorisvandenbossche @pitrou 


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] lidavidm commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
lidavidm commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r848356453


##########
python/pyarrow/compute.py:
##########
@@ -597,16 +597,29 @@ def field(name_or_index):
     Stores only the field's name. Type and other information is known only when
     the expression is bound to a dataset having an explicit scheme.
 
+    Nested references are allowed by passing a tuple of names.
+    For example ``('foo', 'bar')`` references the field named "bar" inside
+    the field named "foo".
+
     Parameters
     ----------
-    name_or_index : string or int
-        The name or index of the field the expression references to.
+    name_or_index : string, tuple or int

Review Comment:
   I wonder if we should also allow dotted names (though that could get ambiguous), or else, allow `field("a", "b")` instead of requiring `field(("a", "b"))`. That said neither are essential here.



##########
python/pyarrow/tests/test_dataset.py:
##########
@@ -391,6 +393,14 @@ def test_dataset(dataset, dataset_reader):
     assert sorted(result['group']) == [1, 2]
     assert sorted(result['key']) == ['xxx', 'yyy']
 
+    condition = ds.field(('struct', 'b')) == '1'
+    result = dataset.to_table(use_threads=True, filter=condition).to_pydict()

Review Comment:
   Can we test a column projection as well somewhere here?



##########
python/pyarrow/_compute.pyx:
##########
@@ -2219,10 +2219,26 @@ cdef class Expression(_Weakrefable):
 
     @staticmethod
     def _field(name_or_idx not None):
-        if isinstance(name_or_idx, str):
-            return Expression.wrap(CMakeFieldExpression(tobytes(name_or_idx)))
-        else:
+        cdef:
+            CFieldRef c_field
+
+        if isinstance(name_or_idx, int):
             return Expression.wrap(CMakeFieldExpressionByIndex(name_or_idx))
+        else:
+            c_field = CFieldRef(<c_string> tobytes(name_or_idx))
+            return Expression.wrap(CMakeFieldExpression(c_field))
+
+    @staticmethod
+    def _nested_field(tuple names not None):
+        cdef:
+            vector[CFieldRef] nested
+
+        if len(names) == 0:
+            raise ValueError("nested field reference should be non-empty")
+        nested.reserve(len(names))
+        for name in names:
+            nested.push_back(CFieldRef(<c_string> tobytes(name)))

Review Comment:
   We might want to file a follow up for nested field refs that use indices? ARROW-15658/#12664 are working on support for 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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r848476086


##########
cpp/src/arrow/compute/exec/expression_test.cc:
##########
@@ -1376,12 +1376,18 @@ TEST(Expression, SerializationRoundTrips) {
 
   ExpectRoundTrips(field_ref("field"));
 
+  ExpectRoundTrips(field_ref(FieldRef("foo", "bar", "baz")));

Review Comment:
   Will try to simplify.



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] pitrou closed pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
pitrou closed pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field
URL: https://github.com/apache/arrow/pull/12863


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r848475601


##########
python/pyarrow/compute.py:
##########
@@ -597,16 +597,29 @@ def field(name_or_index):
     Stores only the field's name. Type and other information is known only when
     the expression is bound to a dataset having an explicit scheme.
 
+    Nested references are allowed by passing a tuple of names.
+    For example ``('foo', 'bar')`` references the field named "bar" inside
+    the field named "foo".
+
     Parameters
     ----------
-    name_or_index : string or int
-        The name or index of the field the expression references to.
+    name_or_index : string, tuple or int
+        The name or index of the (possibly nested) field the expression
+        references to.
 
     Returns
     -------
     field_expr : Expression
     """
-    return Expression._field(name_or_index)
+    if isinstance(name_or_index, str) or isinstance(name_or_index, int):

Review Comment:
   🤩 



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r849061122


##########
python/pyarrow/compute.py:
##########
@@ -597,16 +597,29 @@ def field(name_or_index):
     Stores only the field's name. Type and other information is known only when
     the expression is bound to a dataset having an explicit scheme.
 
+    Nested references are allowed by passing a tuple of names.
+    For example ``('foo', 'bar')`` references the field named "bar" inside
+    the field named "foo".
+
     Parameters
     ----------
-    name_or_index : string or int
-        The name or index of the field the expression references to.
+    name_or_index : string, tuple or int

Review Comment:
   I think that allowing `field("a", "b")` would make a lot of sense. Will try to implement it here and if I run into difficulties I will add a follow-up JIRA.



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r852186238


##########
python/pyarrow/tests/test_dataset.py:
##########
@@ -391,6 +393,26 @@ def test_dataset(dataset, dataset_reader):
     assert sorted(result['group']) == [1, 2]
     assert sorted(result['key']) == ['xxx', 'yyy']
 
+    condition = ds.field(('struct', 'b')) == '1'
+    result = dataset.to_table(use_threads=True, filter=condition).to_pydict()
+
+    assert result['i64'] == [1, 4, 1, 4]

Review Comment:
   🤦‍♀️🤷‍♀️ Thank you!



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] github-actions[bot] commented on pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #12863:
URL: https://github.com/apache/arrow/pull/12863#issuecomment-1096463815

   https://issues.apache.org/jira/browse/ARROW-11259


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] pitrou commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
pitrou commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r848416435


##########
cpp/src/arrow/compute/exec/expression_test.cc:
##########
@@ -1376,12 +1376,18 @@ TEST(Expression, SerializationRoundTrips) {
 
   ExpectRoundTrips(field_ref("field"));
 
+  ExpectRoundTrips(field_ref(FieldRef("foo", "bar", "baz")));

Review Comment:
   Do we have to spell out `field_ref(FieldRef("foo", "bar"))` explicitly or is it possible to simply write `field_ref("foo", "bar")`?



##########
python/pyarrow/compute.py:
##########
@@ -597,16 +597,29 @@ def field(name_or_index):
     Stores only the field's name. Type and other information is known only when
     the expression is bound to a dataset having an explicit scheme.
 
+    Nested references are allowed by passing a tuple of names.
+    For example ``('foo', 'bar')`` references the field named "bar" inside
+    the field named "foo".
+
     Parameters
     ----------
-    name_or_index : string or int
-        The name or index of the field the expression references to.
+    name_or_index : string, tuple or int
+        The name or index of the (possibly nested) field the expression
+        references to.
 
     Returns
     -------
     field_expr : Expression
     """
-    return Expression._field(name_or_index)
+    if isinstance(name_or_index, str) or isinstance(name_or_index, int):

Review Comment:
   Nit :-)
   ```suggestion
       if isinstance(name_or_index, (str, int)):
   ```



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] pitrou commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
pitrou commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r850464777


##########
python/pyarrow/tests/test_dataset.py:
##########
@@ -391,6 +393,26 @@ def test_dataset(dataset, dataset_reader):
     assert sorted(result['group']) == [1, 2]
     assert sorted(result['key']) == ['xxx', 'yyy']
 
+    condition = ds.field(('struct', 'b')) == '1'
+    result = dataset.to_table(use_threads=True, filter=condition).to_pydict()
+
+    assert result['i64'] == [1, 4, 1, 4]

Review Comment:
   This one could be in another order, you should probably sort the result table before (using `Table.sort_by` perhaps).



##########
python/pyarrow/tests/test_dataset.py:
##########
@@ -391,6 +393,26 @@ def test_dataset(dataset, dataset_reader):
     assert sorted(result['group']) == [1, 2]
     assert sorted(result['key']) == ['xxx', 'yyy']
 
+    condition = ds.field(('struct', 'b')) == '1'
+    result = dataset.to_table(use_threads=True, filter=condition).to_pydict()
+
+    assert result['i64'] == [1, 4, 1, 4]
+    assert result['f64'] == [1.0, 4.0, 1.0, 4.0]
+    assert sorted(result['group']) == [1, 1, 2, 2]
+    assert sorted(result['key']) == ['xxx', 'xxx', 'yyy', 'yyy']
+
+    projection = {
+        'i64': ds.field('i64'),
+        'f64': ds.field('f64'),
+        "new": ds.field(('struct', 'b')) == '1',
+    }
+    result = dataset.to_table(use_threads=True, columns=projection).to_pydict()
+
+    assert result['i64'] == [0, 1, 2, 3, 4, 0, 1, 2, 3, 4]

Review Comment:
   Same here (sort the results)?



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r849060220


##########
python/pyarrow/_compute.pyx:
##########
@@ -2219,10 +2219,26 @@ cdef class Expression(_Weakrefable):
 
     @staticmethod
     def _field(name_or_idx not None):
-        if isinstance(name_or_idx, str):
-            return Expression.wrap(CMakeFieldExpression(tobytes(name_or_idx)))
-        else:
+        cdef:
+            CFieldRef c_field
+
+        if isinstance(name_or_idx, int):
             return Expression.wrap(CMakeFieldExpressionByIndex(name_or_idx))
+        else:
+            c_field = CFieldRef(<c_string> tobytes(name_or_idx))
+            return Expression.wrap(CMakeFieldExpression(c_field))
+
+    @staticmethod
+    def _nested_field(tuple names not None):
+        cdef:
+            vector[CFieldRef] nested
+
+        if len(names) == 0:
+            raise ValueError("nested field reference should be non-empty")
+        nested.reserve(len(names))
+        for name in names:
+            nested.push_back(CFieldRef(<c_string> tobytes(name)))

Review Comment:
   https://issues.apache.org/jira/browse/ARROW-16180



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on PR #12863:
URL: https://github.com/apache/arrow/pull/12863#issuecomment-1098738984

   Thank you for the comments @lidavidm @pitrou!
   I _think_ this PR is ready for another round of review (the errors should be related to the S3fs) .


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r850530904


##########
python/pyarrow/tests/test_dataset.py:
##########
@@ -391,6 +393,26 @@ def test_dataset(dataset, dataset_reader):
     assert sorted(result['group']) == [1, 2]
     assert sorted(result['key']) == ['xxx', 'yyy']
 
+    condition = ds.field(('struct', 'b')) == '1'
+    result = dataset.to_table(use_threads=True, filter=condition).to_pydict()
+
+    assert result['i64'] == [1, 4, 1, 4]

Review Comment:
   Sure, will do 👍 



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r848482152


##########
python/pyarrow/tests/test_dataset.py:
##########
@@ -391,6 +393,14 @@ def test_dataset(dataset, dataset_reader):
     assert sorted(result['group']) == [1, 2]
     assert sorted(result['key']) == ['xxx', 'yyy']
 
+    condition = ds.field(('struct', 'b')) == '1'
+    result = dataset.to_table(use_threads=True, filter=condition).to_pydict()

Review Comment:
   Sure, will do 👍 



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] pitrou commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
pitrou commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r851999609


##########
python/pyarrow/tests/test_dataset.py:
##########
@@ -391,6 +393,26 @@ def test_dataset(dataset, dataset_reader):
     assert sorted(result['group']) == [1, 2]
     assert sorted(result['key']) == ['xxx', 'yyy']
 
+    condition = ds.field(('struct', 'b')) == '1'
+    result = dataset.to_table(use_threads=True, filter=condition).to_pydict()
+
+    assert result['i64'] == [1, 4, 1, 4]

Review Comment:
   Ok, I pushed a commit to show better what I meant :-)



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] github-actions[bot] commented on pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #12863:
URL: https://github.com/apache/arrow/pull/12863#issuecomment-1096463849

   :warning: Ticket **has not been started in JIRA**, please click 'Start Progress'.


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] AlenkaF commented on a diff in pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
AlenkaF commented on code in PR #12863:
URL: https://github.com/apache/arrow/pull/12863#discussion_r849220598


##########
cpp/src/arrow/compute/exec/expression_test.cc:
##########
@@ -1376,12 +1376,18 @@ TEST(Expression, SerializationRoundTrips) {
 
   ExpectRoundTrips(field_ref("field"));
 
+  ExpectRoundTrips(field_ref(FieldRef("foo", "bar", "baz")));

Review Comment:
   Currently it doesn't work, `FieldRef("foo", "bar")` needs to be spelled out.
   
   <details>
   <summary>Try 1: `field_ref("foo", "bar")`</summary>
   <pre>
   ```
   /Users/alenkafrim/repos/arrow/cpp/src/arrow/compute/exec/expression_test.cc:1379:20: error: no matching function for call to 'field_ref'
     ExpectRoundTrips(field_ref("foo", "bar", "baz"));
                      ^~~~~~~~~
   /Users/alenkafrim/repos/arrow/cpp/src/arrow/compute/exec/expression.h:152:12: note: candidate function not viable: requires single argument 'ref', but 3 arguments were provided
   Expression field_ref(FieldRef ref);
   ```
   </pre>
   </details>
   
   <details>
   <summary>Try 2: `field_ref(("foo", "bar"))`</summary>
   <pre>
   ```
   /Users/alenkafrim/repos/arrow/cpp/src/arrow/compute/exec/expression_test.cc:1379:31: warning: expression result unused [-Wunused-value]
     ExpectRoundTrips(field_ref(("foo", "bar", "baz")));
   ```
   </pre>
   </details>



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] ursabot commented on pull request #12863: ARROW-11259: [Python] Allow to create field reference to nested field

Posted by GitBox <gi...@apache.org>.
ursabot commented on PR #12863:
URL: https://github.com/apache/arrow/pull/12863#issuecomment-1101411121

   Benchmark runs are scheduled for baseline = 83dda5c312238a2ce6e4d553e689444a8f665606 and contender = e0668e2024a7929abf2b0d8b53d2886a2e3a3379. e0668e2024a7929abf2b0d8b53d2886a2e3a3379 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
   Conbench compare runs links:
   [Finished :arrow_down:0.0% :arrow_up:0.0%] [ec2-t3-xlarge-us-east-2](https://conbench.ursa.dev/compare/runs/f6fc15a3fec745749234fdf7df3e19bb...bddad4a507804919b8f8c707aa51ade0/)
   [Failed :arrow_down:0.5% :arrow_up:0.04%] [test-mac-arm](https://conbench.ursa.dev/compare/runs/6c2cf55ff1464790b003fa11f63af581...a462410cd9774e2f9cfbeda01817fc30/)
   [Failed :arrow_down:0.0% :arrow_up:0.0%] [ursa-i9-9960x](https://conbench.ursa.dev/compare/runs/73d122407e934a1ca2e28862ce68e732...f9df59d6f4ff4650a6d8ce1139ff8cb5/)
   [Finished :arrow_down:0.26% :arrow_up:0.0%] [ursa-thinkcentre-m75q](https://conbench.ursa.dev/compare/runs/b96295f65a47420da4f6259320c4328a...e3cc6be67f40436b8929cb61d714b81b/)
   Buildkite builds:
   [Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/524| `e0668e20` ec2-t3-xlarge-us-east-2>
   [Failed] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/511| `e0668e20` test-mac-arm>
   [Failed] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/510| `e0668e20` ursa-i9-9960x>
   [Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/521| `e0668e20` ursa-thinkcentre-m75q>
   [Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/523| `83dda5c3` ec2-t3-xlarge-us-east-2>
   [Failed] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/510| `83dda5c3` test-mac-arm>
   [Failed] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/509| `83dda5c3` ursa-i9-9960x>
   [Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/520| `83dda5c3` ursa-thinkcentre-m75q>
   Supported benchmarks:
   ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
   test-mac-arm: Supported benchmark langs: C++, Python, R
   ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
   ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java
   


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org