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/05/18 18:22:11 UTC

[GitHub] [arrow] pitrou opened a new pull request #7216: ARROW-7778: [Integration][C++] Enable nested dictionaries

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


   


----------------------------------------------------------------
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] wesm closed pull request #7216: ARROW-7778: [Integration][C++] Enable nested dictionaries

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


   


----------------------------------------------------------------
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 edited a comment on pull request #7216: ARROW-7778: [Integration][C++] Enable nested dictionaries

Posted by GitBox <gi...@apache.org>.
pitrou edited a comment on pull request #7216:
URL: https://github.com/apache/arrow/pull/7216#issuecomment-631523683


   @wesm I would like your high-level opinion on this. With this PR, we can decode nested dictionaries provided that inner dictionaries are serialized before outer dictionaries on the IPC stream. This seems like a reasonable requirement to me, but of course it isn't spelled out in the format spec.


----------------------------------------------------------------
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] github-actions[bot] commented on pull request #7216: ARROW-7778: [Integration][C++] Enable nested dictionaries

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


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


----------------------------------------------------------------
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] wesm commented on pull request #7216: ARROW-7778: [Integration][C++] Enable nested dictionaries

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


   Let's bring it up on the ML. Seems a reasonable requirement to me. 


----------------------------------------------------------------
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 #7216: ARROW-7778: [Integration][C++] Enable nested dictionaries

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


   @wesm I would like your high-level opinion on this. With this PR, we can decode nested dictionaries if inner dictionaries are serialized before outer dictionaries on the IPC stream. This seems like a reasonable requirement to me, but of course it isn't spelled out in the format spec.


----------------------------------------------------------------
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 #7216: ARROW-7778: [Integration][C++] Enable nested dictionaries

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


   Or should I ask on the ML?


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