parquet does not compile with features=["zstd"]

[alamb]

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>

arrow-rs/parquet/Cargo.toml

Line 41 in 7da41c7

chrono = { version = "0.4", default-features = false }

This should not have default-features = false

Originally posted by @zeroflaw in #1623 (comment)

[alamb]

One way to work around this is to manually add a dependency on chrono with the correct features, such as

chrono = { version = "0.4" }
parquet = { version = "12.0.0", default-features = false, features = ["zstd"] }

[zeroflaw]

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.

[alamb]

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 #1000 by @carols10cents

More investigation is required to figure out the right fix I think

[asayers]

FWIW I've had this in my Cargo.toml since parquet-0.9:

# 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"] }