Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693080874 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1466 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
liurenjie1024 merged PR #475: URL: https://github.com/apache/iceberg-rust/pull/475 -- 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: issues-unsubscr...@ic

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
liurenjie1024 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693113401 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1466 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
liurenjie1024 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693082893 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693080874 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1466 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
liurenjie1024 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693080828 ## crates/catalog/inmemory/src/namespace_state.rs: ## @@ -0,0 +1,306 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693076550 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
liurenjie1024 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693073031 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
liurenjie1024 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693069613 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1466 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693069969 ## crates/catalog/inmemory/src/namespace_state.rs: ## @@ -0,0 +1,306 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor lice

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693068061 ## crates/catalog/inmemory/Cargo.toml: ## @@ -0,0 +1,41 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements.

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1693048009 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1466 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
liurenjie1024 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1689983750 ## crates/catalog/inmemory/src/lib.rs: ## @@ -0,0 +1,26 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agree

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-26 Thread via GitHub
liurenjie1024 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1689983750 ## crates/catalog/inmemory/src/lib.rs: ## @@ -0,0 +1,26 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agree

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691991685 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691909039 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691932105 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691909039 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691909039 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691852790 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691852790 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691852790 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691830743 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691830743 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691830743 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691637775 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691637775 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691637775 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691550143 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691550143 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691550143 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691550143 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691550143 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691464398 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691483403 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-25 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1691464398 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690861564 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1496 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreem

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
Xuanwo commented on PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#issuecomment-2249346483 Request review from @Fokko and @liurenjie1024 for another eyes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690620132 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1511 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
fqaiser94 commented on PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#issuecomment-2249126369 @Xuanwo just FYI, all comments besides the [package-naming](https://github.com/apache/iceberg-rust/pull/475#discussion_r1690026333) discussion should be addressed now -- This is

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690627063 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1511 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690621507 ## crates/catalog/inmemory/src/lib.rs: ## @@ -0,0 +1,26 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreement

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690620132 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1511 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690616688 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1511 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
fqaiser94 commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690614275 ## crates/catalog/inmemory/src/catalog.rs: ## @@ -0,0 +1,1511 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agr

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690026333 ## crates/catalog/inmemory/Cargo.toml: ## @@ -0,0 +1,41 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. S

Re: [PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-24 Thread via GitHub
Xuanwo commented on code in PR #475: URL: https://github.com/apache/iceberg-rust/pull/475#discussion_r1690027985 ## crates/catalog/inmemory/README.md: ## @@ -0,0 +1,27 @@ + + +# Apache Iceberg In-Memory Catalog Official Native Rust Implementation + +[![crates.io](https://img.shi

[PR] Add in-memory catalog implementation [iceberg-rust]

2024-07-23 Thread via GitHub
fqaiser94 opened a new pull request, #475: URL: https://github.com/apache/iceberg-rust/pull/475 Fixes: https://github.com/apache/iceberg-rust/issues/412 This PR adds an in-memory implementation of catalog. Given that this is in-memory, it's not suitable for use in production e