iceberg-rust icon indicating copy to clipboard operation
iceberg-rust copied to clipboard

Published 20 hours ago • verified publisher icon apache

docs: installation of the new `iceberg_catalog_rest` added to the docs

Open nishant-sachdeva opened this issue 10 months ago • 5 comments

nishant-sachdeva avatar Apr 27 '24 08:04 nishant-sachdeva

Also cc @Fokko and @liurenjie1024 for a review.

Xuanwo avatar Apr 27 '24 08:04 Xuanwo

cc @nishant-sachdeva Are you still working on this?

liurenjie1024 avatar May 22 '24 14:05 liurenjie1024

Hi, @liurenjie1024, this PR has been open for a while without any progress. I believe it's ready to be merged and we can continue to make improvements on it afterward. What do you think about merging it as is?

Xuanwo avatar Jul 25 '24 10:07 Xuanwo

Hi, @liurenjie1024, this PR has been open for a while without any progress. I believe it's ready to be merged and we can continue to make improvements on it afterward. What do you think about merging it as is?

Seems @nishant-sachdeva is not responding. To be honest I still prefer to use different section for adding different dependencies. Or could we file other prs to add docs and close this pr first if @nishant-sachdeva doesn't have time to do the udpate?

liurenjie1024 avatar Jul 26 '24 02:07 liurenjie1024

Seems @nishant-sachdeva is not responding. To be honest I still prefer to use different section for adding different dependencies.

Understood.

Or could we file other prs to add docs and close this pr first if @nishant-sachdeva doesn't have time to do the udpate?

@nishant-sachdeva has made some effort in this PR and I do want to include his contribution in. I will finish this work based on his work on this PR.

Xuanwo avatar Jul 26 '24 04:07 Xuanwo