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/04/29 17:13:22 UTC

[GitHub] [arrow-rs] alamb opened a new issue, #1630: parquet does not compile with `features=["zstd"]`

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

   reported by @zeroflaw
   
   
   A project with 
   
   `parquet = { version = "12.0.0", default-features = false, features = ["zstd"] }`
   
   Fails to compile with
   
   `format!("{}", dt.format("%Y-%m-%d %H:%M:%S %:z"))
       |                      ^^^^^^ method not found in DateTime<Utc>`
   
   https://github.com/apache/arrow-rs/blob/7da41c7328820241a1e9a06eb83cade2e1f08da4/parquet/Cargo.toml#L41
   
   This should not have default-features = false
   
   _Originally posted by @zeroflaw in https://github.com/apache/arrow-rs/issues/1623#issuecomment-1113196552_


-- 
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 closed issue #1630: parquet does not compile with `features=["zstd"]`

Posted by GitBox <gi...@apache.org>.
tustvold closed issue #1630: parquet does not compile with `features=["zstd"]`
URL: https://github.com/apache/arrow-rs/issues/1630


-- 
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] asayers commented on issue #1630: parquet does not compile with `features=["zstd"]`

Posted by GitBox <gi...@apache.org>.
asayers commented on issue #1630:
URL: https://github.com/apache/arrow-rs/issues/1630#issuecomment-1128274044

   FWIW I've had this in my Cargo.toml since parquet-0.9:
   
   ```toml
   # Chrono is not a direct dependency; it's a dependency of "parquet".  However,
   # parquet's Cargo.toml is slightly buggy, so if we don't specify chrono here
   # then parquet fails to compile.
   chrono = { version = "0.4.19", default-features = false, features = ["clock"] }
   ```


-- 
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] zeroflaw commented on issue #1630: parquet does not compile with `features=["zstd"]`

Posted by GitBox <gi...@apache.org>.
zeroflaw commented on issue #1630:
URL: https://github.com/apache/arrow-rs/issues/1630#issuecomment-1113557531

   Thats what I did, but it was not required in version 11. 
   If a compile error is expected, then i'm happy with the solution. 


-- 
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 #1630: parquet does not compile with `features=["zstd"]`

Posted by GitBox <gi...@apache.org>.
alamb commented on issue #1630:
URL: https://github.com/apache/arrow-rs/issues/1630#issuecomment-1113546830

   One way to work around this is to manually add a dependency on `chrono` with the correct features, such as
   
   ```toml
   chrono = { version = "0.4" }
   parquet = { version = "12.0.0", default-features = false, features = ["zstd"] }
   ```


-- 
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 #1630: parquet does not compile with `features=["zstd"]`

Posted by GitBox <gi...@apache.org>.
alamb commented on issue #1630:
URL: https://github.com/apache/arrow-rs/issues/1630#issuecomment-1113563274

   > If a compile error is expected, then i'm happy with the solution.
   
   No I don't think a compile error is expected. This definitely seems like a bug -- I just wanted to document the issue (and solution) in case someone else ran into it as well
   
   I am not sure removing `default-features=false` is the right fix -- I think that was changed to minimize the size of the arrow dependency stack -- done in https://github.com/apache/arrow-rs/pull/1000 by @carols10cents 
   
   More investigation is required to figure out the right fix I think


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