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 2020/12/15 12:26:13 UTC

[GitHub] [arrow] pitrou opened a new pull request #8925: [Doc] Update status matrix for Decimal256

pitrou opened a new pull request #8925:
URL: https://github.com/apache/arrow/pull/8925


   


----------------------------------------------------------------
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.

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



[GitHub] [arrow] pitrou closed pull request #8925: ARROW-11124: [Doc] Update status matrix for Decimal256

Posted by GitBox <gi...@apache.org>.
pitrou closed pull request #8925:
URL: https://github.com/apache/arrow/pull/8925


   


----------------------------------------------------------------
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.

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



[GitHub] [arrow] quinnj commented on a change in pull request #8925: [Doc] Update status matrix for Decimal256

Posted by GitBox <gi...@apache.org>.
quinnj commented on a change in pull request #8925:
URL: https://github.com/apache/arrow/pull/8925#discussion_r543514355



##########
File path: docs/source/status.rst
##########
@@ -44,6 +44,8 @@ Data Types
 +-------------------+-------+-------+-------+------------+-------+-------+-------+
 | Decimal128        | ✓     | ✓     |       |            |       |       | ✓     |
 +-------------------+-------+-------+-------+------------+-------+-------+-------+
+| Decimal256        | ✓     | ✓     |       |            |       |       |       |

Review comment:
       Not yet, but I started the work locally, just need to find time to finish it up. It should be pretty easy to at least allow deserializing/serializing, but more work to make the Decimal256 useful for arithmetic, etc.




----------------------------------------------------------------
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.

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



[GitHub] [arrow] pitrou commented on pull request #8925: [Doc] Update status matrix for Decimal256

Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #8925:
URL: https://github.com/apache/arrow/pull/8925#issuecomment-745256597


   cc @emkornfield 


----------------------------------------------------------------
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.

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



[GitHub] [arrow] pitrou commented on a change in pull request #8925: [Doc] Update status matrix for Decimal256

Posted by GitBox <gi...@apache.org>.
pitrou commented on a change in pull request #8925:
URL: https://github.com/apache/arrow/pull/8925#discussion_r543299488



##########
File path: docs/source/status.rst
##########
@@ -44,6 +44,8 @@ Data Types
 +-------------------+-------+-------+-------+------------+-------+-------+-------+
 | Decimal128        | ✓     | ✓     |       |            |       |       | ✓     |
 +-------------------+-------+-------+-------+------------+-------+-------+-------+
+| Decimal256        | ✓     | ✓     |       |            |       |       |       |

Review comment:
       @quinnj Is Decimal256 supported on Julia?




----------------------------------------------------------------
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.

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



[GitHub] [arrow] pitrou commented on pull request #8925: [Doc] Update status matrix for Decimal256

Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #8925:
URL: https://github.com/apache/arrow/pull/8925#issuecomment-745256496


   A JIRA is probably not needed for this.


----------------------------------------------------------------
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.

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