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/11/14 12:28:43 UTC

[GitHub] [arrow] rok opened a new pull request, #14632: MINOR: [DOCS][C++] Improve docs for ConcatenateTables

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

   This is to improve [ConcatenateTables docs](https://github.com/apache/arrow/blob/9d86ede69af14eacc7968684b7e9535cfc8e3907/cpp/src/arrow/table.h#L291).


-- 
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] wjones127 commented on a diff in pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,23 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables.
+///
+/// The new table is assembled from existing column chunks without copying,
+/// if schemas are identical. If schemas do not match exactly, an attempt is
+/// made to unify them, and then column chunks are converted to their
+/// respective unified datatype, which will probably incur a copy.
+/// :func:`arrow::PromoteTableToSchema` is used to unify schemas.

Review Comment:
   Schemas will only be unified if requested.
   
   ```suggestion
   /// if schemas are identical. If schemas do not match exactly and 
   /// unifySchemas is enabled in options (off by default), an attempt is
   /// made to unify them, and then column chunks are converted to their
   /// respective unified datatype, which will probably incur a copy.
   /// :func:`arrow::PromoteTableToSchema` is used to unify schemas.
   ```



-- 
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] rok commented on a diff in pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,22 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables.
+///
+/// The new table is assembled from existing column chunks without copying,
+/// if schemas are identical. If schemas do not match exactly, an attempt is
+/// made to unify them, and then column chunks are converted to their

Review Comment:
   Added.



-- 
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 #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,18 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables. Underlying column arrays
+/// are concatenated without copying except in case of schema missmatch in which case
+/// columns will be cast to promoted types which will cause a copy. Tables will be
+/// concatenated in order they are provided in and order of rows within tables will be
+/// preserved.
+///
+/// \param[in] tables a std::vector of Tables to be concatenated
+/// \param[in] options specify how to unify schema of input tables
+/// \param[in] memory_pool MemoryPool to be used if null-filled arrays need to
+/// be created.
+/// \return Status

Review Comment:
   Hmm... no? :-)



-- 
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] wjones127 commented on a diff in pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,23 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables.
+///
+/// The new table is assembled from existing column chunks without copying,
+/// if schemas are identical. If schemas do not match exactly, an attempt is
+/// made to unify them, and then column chunks are converted to their
+/// respective unified datatype, which will probably incur a copy.
+/// :func:`arrow::PromoteTableToSchema` is used to unify schemas.

Review Comment:
   Schemas will only be unified if requested.
   
   ```suggestion
   /// if schemas are identical. If schemas do not match exactly and 
   /// unify_schemas is enabled in options (off by default), an attempt is
   /// made to unify them, and then column chunks are converted to their
   /// respective unified datatype, which will probably incur a copy.
   /// :func:`arrow::PromoteTableToSchema` is used to unify schemas.
   ```



-- 
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] rok commented on a diff in pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,18 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables. Underlying column arrays
+/// are concatenated without copying except in case of schema missmatch in which case
+/// columns will be cast to promoted types which will cause a copy. Tables will be
+/// concatenated in order they are provided in and order of rows within tables will be
+/// preserved.
+///
+/// \param[in] tables a std::vector of Tables to be concatenated
+/// \param[in] options specify how to unify schema of input tables
+/// \param[in] memory_pool MemoryPool to be used if null-filled arrays need to
+/// be created.
+/// \return Status

Review Comment:
   Ugh sorry.



-- 
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 #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,18 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables. Underlying column arrays
+/// are concatenated without copying except in case of schema missmatch in which case
+/// columns will be cast to promoted types which will cause a copy. Tables will be
+/// concatenated in order they are provided in and order of rows within tables will be
+/// preserved.
+///
+/// \param[in] tables a std::vector of Tables to be concatenated
+/// \param[in] options specify how to unify schema of input tables
+/// \param[in] memory_pool MemoryPool to be used if null-filled arrays need to
+/// be created.

Review Comment:
   ```suggestion
   /// \param[in] memory_pool MemoryPool to be used if null-filled arrays need to
   /// be created or if existing column chunks need to be endure type conversion
   ```



-- 
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] rok commented on a diff in pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,22 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables.
+///
+/// The new table is assembled from existing column chunks without copying,
+/// if schemas are identical. If schemas do not match exactly, an attempt is
+/// made to unify them, and then column chunks are converted to their

Review Comment:
   @pitrou could docs of other functions be referenced from docstrings? E.g. it would make sense to link to docs of [PromoteTableToSchema](https://github.com/apache/arrow/blob/9d86ede69af14eacc7968684b7e9535cfc8e3907/cpp/src/arrow/table.h#L296-L310). Also `PromoteTableToSchema` docs doesn't seem to be public. Is that ideal?



-- 
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] rok commented on a diff in pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,18 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables. Underlying column arrays
+/// are concatenated without copying except in case of schema missmatch in which case
+/// columns will be cast to promoted types which will cause a copy. Tables will be
+/// concatenated in order they are provided in and order of rows within tables will be
+/// preserved.
+///
+/// \param[in] tables a std::vector of Tables to be concatenated
+/// \param[in] options specify how to unify schema of input tables
+/// \param[in] memory_pool MemoryPool to be used if null-filled arrays need to
+/// be created.
+/// \return Status

Review Comment:
   ```suggestion
   /// \return new Table
   ```



-- 
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] jorisvandenbossche commented on a diff in pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,24 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables.
+///
+/// The new table is assembled from existing column chunks without copying,
+/// if schemas are identical. If schemas do not match exactly and
+/// unifySchemas is enabled in options (off by default), an attempt is

Review Comment:
   ```suggestion
   /// unify_schemas is enabled in options (off by default), an attempt is
   ```



-- 
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 #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,18 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables. Underlying column arrays
+/// are concatenated without copying except in case of schema missmatch in which case
+/// columns will be cast to promoted types which will cause a copy. Tables will be
+/// concatenated in order they are provided in and order of rows within tables will be
+/// preserved.

Review Comment:
   We should keep the brief line brief. I'd also like to suggest some wording changes.
   
   ```suggestion
   /// \brief Construct a new table from multiple input tables.
   ///
   /// The new table is assembled from existing column chunks without copying,
   /// if schemas are identical. If schemas do not match exactly, an attempt is
   /// made to unify them, and then column chunks are converted to their
   /// respective unified datatype, which will probably incur a copy.
   ///
   /// Tables are concatenated in order they are provided in and the order of
   /// rows within tables will be preserved.
   ```
   



-- 
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 #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,18 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables. Underlying column arrays
+/// are concatenated without copying except in case of schema missmatch in which case
+/// columns will be cast to promoted types which will cause a copy. Tables will be
+/// concatenated in order they are provided in and order of rows within tables will be
+/// preserved.
+///
+/// \param[in] tables a std::vector of Tables to be concatenated
+/// \param[in] options specify how to unify schema of input tables
+/// \param[in] memory_pool MemoryPool to be used if null-filled arrays need to
+/// be created.

Review Comment:
   ```suggestion
   /// \param[in] memory_pool MemoryPool to be used if null-filled arrays need to
   /// be created or if existing column chunks need to endure type conversion
   ```



-- 
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 #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,22 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables.
+///
+/// The new table is assembled from existing column chunks without copying,
+/// if schemas are identical. If schemas do not match exactly, an attempt is
+/// made to unify them, and then column chunks are converted to their

Review Comment:
   Yes, you can reference `PromoteTableToSchema` here. And, yes, we can publish the doc to the latter in the Sphinx API docs.



-- 
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] rok commented on pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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

   Docs changes should be [here](http://crossbow.voltrondata.com/pr_docs/14632/cpp/api/table.html#_CPPv4N5arrow17ConcatenateTablesERKNSt6vectorINSt10shared_ptrI5TableEEEE24ConcatenateTablesOptionsP10MemoryPool) once built.


-- 
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 merged pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

Posted by GitBox <gi...@apache.org>.
pitrou merged PR #14632:
URL: https://github.com/apache/arrow/pull/14632


-- 
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] rok commented on pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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

   Thanks all! :)


-- 
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] rok commented on a diff in pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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


##########
cpp/src/arrow/table.h:
##########
@@ -286,7 +286,24 @@ struct ARROW_EXPORT ConcatenateTablesOptions {
   static ConcatenateTablesOptions Defaults() { return {}; }
 };
 
-/// \brief Construct table from multiple input tables.
+/// \brief Construct a new table from multiple input tables.
+///
+/// The new table is assembled from existing column chunks without copying,
+/// if schemas are identical. If schemas do not match exactly and 

Review Comment:
   ```suggestion
   /// if schemas are identical. If schemas do not match exactly and
   ```



-- 
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 #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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

   Revision: 6e52fb4e9a51eaa3b6ea38061f7b2c25488558ea
   
   Submitted crossbow builds: [ursacomputing/crossbow @ actions-77fdcac283](https://github.com/ursacomputing/crossbow/branches/all?query=actions-77fdcac283)
   
   |Task|Status|
   |----|------|
   |preview-docs|[![Github Actions](https://github.com/ursacomputing/crossbow/workflows/Crossbow/badge.svg?branch=actions-77fdcac283-github-preview-docs)](https://github.com/ursacomputing/crossbow/actions/runs/3462393759/jobs/5781223999)|


-- 
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] rok commented on pull request #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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

   @github-actions crossbow submit preview-docs


-- 
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 #14632: MINOR: [Docs][C++] Improve docs for ConcatenateTables

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

   Benchmark runs are scheduled for baseline = dab5d3a29394f59045ac6b66f9b697507d6cd1b7 and contender = 4daf94587d2aac1ee23cea6e94cc99a06512f84a. 4daf94587d2aac1ee23cea6e94cc99a06512f84a 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/9bf58ea56f7240f38217667d2ba71fc0...f6d1d58aab834369a92d81cc7f9b214c/)
   [Finished :arrow_down:0.24% :arrow_up:0.0%] [test-mac-arm](https://conbench.ursa.dev/compare/runs/811830279f7449fe8beb11e3e2e98988...07a5f38130044fc7bbe9afa93869e88c/)
   [Finished :arrow_down:0.0% :arrow_up:0.0%] [ursa-i9-9960x](https://conbench.ursa.dev/compare/runs/328f226a0cee444dbffe9b4759fe3262...a088651a685c4d5ea27dd83191d578a3/)
   [Finished :arrow_down:0.28% :arrow_up:0.0%] [ursa-thinkcentre-m75q](https://conbench.ursa.dev/compare/runs/f1a6197d7058415bac6ee89325827699...8bff798b77c4414dad5985d3d350e2c1/)
   Buildkite builds:
   [Finished] [`4daf9458` ec2-t3-xlarge-us-east-2](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/1865)
   [Finished] [`4daf9458` test-mac-arm](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/1887)
   [Finished] [`4daf9458` ursa-i9-9960x](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/1854)
   [Finished] [`4daf9458` ursa-thinkcentre-m75q](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/1879)
   [Finished] [`dab5d3a2` ec2-t3-xlarge-us-east-2](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/1864)
   [Finished] [`dab5d3a2` test-mac-arm](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/1886)
   [Finished] [`dab5d3a2` ursa-i9-9960x](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/1853)
   [Finished] [`dab5d3a2` ursa-thinkcentre-m75q](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/1878)
   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