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 2023/09/05 19:53:05 UTC

[GitHub] [arrow-rs] alamb opened a new issue, #4780: `Datum` is not exported as part of `arrow` (it is only exported in `arrow_array`)

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

   Very similar to https://github.com/apache/arrow-rs/issues/4779 but I think this is more serious
   
   **Describe the bug**
   The newly introduced `Datum` is not exported in `arrow` even though it is part of the signature of the new compute kernels such as https://docs.rs/arrow/latest/arrow/compute/kernels/cmp/fn.eq.html
   
   This means that to use some of the functions in the `arrow` crate,  a user has to use `arrow_array` directly, which is inconvenient, or use a deprecated method. 
   
   **To Reproduce**
   Search for `Datum` in arrow: https://docs.rs/arrow/latest/arrow/index.html?search=datum
   
   It only appears in https://docs.rs/arrow-array/46.0.0/arrow_array/trait.Datum.html
   
   **Expected behavior**
   A trait used in the `arrow` crate API should also be exported in that same crate. 
   
   **Additional context**
   Found while updating https://github.com/influxdata/influxdb_iox/pull/8577


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

[GitHub] [arrow-rs] tustvold commented on issue #4780: `Datum` is not exported as part of `arrow` (it is only exported in `arrow_array`)

Posted by "tustvold (via GitHub)" <gi...@apache.org>.
tustvold commented on issue #4780:
URL: https://github.com/apache/arrow-rs/issues/4780#issuecomment-1723616404

   `label_issue.py` automatically added labels {'arrow'} from #4791


-- 
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-rs] alamb commented on issue #4780: `Datum` is not exported as part of `arrow` (it is only exported in `arrow_array`)

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

   > I'm unclear on why this is considered serious
   
   I think it is *more* serious than https://github.com/apache/arrow-rs/issues/4779; I don't think it is particularly serious overall -- more like inconvenient / a rough edge
   
   This means you can't use the new compute kernels for scalars without also importing `arrow_array` which is annoying


-- 
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-rs] tustvold commented on issue #4780: `Datum` is not exported as part of `arrow` (it is only exported in `arrow_array`)

Posted by "tustvold (via GitHub)" <gi...@apache.org>.
tustvold commented on issue #4780:
URL: https://github.com/apache/arrow-rs/issues/4780#issuecomment-1707229015

   I'm unclear on why this is considered serious, most users won't need to reference the trait, but I do agree we should fix 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.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-rs] tustvold closed issue #4780: `Datum` is not exported as part of `arrow` (it is only exported in `arrow_array`)

Posted by "tustvold (via GitHub)" <gi...@apache.org>.
tustvold closed issue #4780: `Datum` is not exported as part of `arrow` (it is only exported in `arrow_array`)
URL: https://github.com/apache/arrow-rs/issues/4780


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