search

apache / iceberg-rust

Apache Iceberg
https://rust.iceberg.apache.org/
Apache License 2.0
469 stars 95 forks source link

idea: RestCatalog::new should not be async #422

Open Xuanwo opened 4 days ago

Xuanwo commented 4 days ago

It's a bit strange for me to find that RestCatalog::new requires async.

How about fetching the token (and update config) when the API is called?

liurenjie1024 commented 3 days ago

LGTM, the logic is similar to java/python, but they have no async function.

Xuanwo commented 3 days ago

Great, let me take it.