You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2024/02/16 14:19:02 UTC

[I] Note that I did not add support for partition_by in proto. We should add a follow up ticket for this. [arrow-datafusion]

alamb opened a new issue, #9248:
URL: https://github.com/apache/arrow-datafusion/issues/9248

                 Note that I did not add support for partition_by in proto. We should add a follow up ticket for this. 
   
   I don't believe this PR will break downstream systems like Ballista's handling of COPY, but it will silently ignore partition_by options until prost is updated.
   
   _Originally posted by @devinjdangelo in https://github.com/apache/arrow-datafusion/pull/9240#discussion_r1491159044_
               


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

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


Re: [I] Add support for `CopyTo::partition_by` field in datafusion_proto [arrow-datafusion]

Posted by "Dandandan (via GitHub)" <gi...@apache.org>.
Dandandan closed issue #9248: Add support for `CopyTo::partition_by` field in datafusion_proto
URL: https://github.com/apache/arrow-datafusion/issues/9248


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


Re: [I] Add support for `CopyTo::partition_by` field in datafusion_proto [arrow-datafusion]

Posted by "PhVHoang (via GitHub)" <gi...@apache.org>.
PhVHoang commented on issue #9248:
URL: https://github.com/apache/arrow-datafusion/issues/9248#issuecomment-1948544683

   I can work on this one. 


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


Re: [I] Add support for `CopyTo::partition_by` field in datafusion_proto [arrow-datafusion]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on issue #9248:
URL: https://github.com/apache/arrow-datafusion/issues/9248#issuecomment-1951871964

   > I can work on this one.
   
   Thanks @PhVHoang  🙏 
   
   https://github.com/apache/arrow-datafusion/pull/9240 was just merged so I think this one is now actionable


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


Re: [I] Add support for `CopyTo::partition_by` field in datafusion_proto [arrow-datafusion]

Posted by "PhVHoang (via GitHub)" <gi...@apache.org>.
PhVHoang commented on issue #9248:
URL: https://github.com/apache/arrow-datafusion/issues/9248#issuecomment-1953556837

   take


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


Re: [I] Add support for `CopyTo::partition_by` field in datafusion_proto [arrow-datafusion]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on issue #9248:
URL: https://github.com/apache/arrow-datafusion/issues/9248#issuecomment-1948480394

   Once https://github.com/apache/arrow-datafusion/pull/9240  is merged, I think this would be a good first issue as it will be small and self contained


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