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 environments. I expect that this will be used mostly for unit testing and maybe demonstration purposes. With this in mind, the implementation prioritizes consistency over performance. For example, all catalog state is stored inside a `Mutex` and each catalog method acquires a lock before performing any operations. This makes all catalog methods thread-safe at the cost of performance. I could get more fancy and look into things like `RWLock` or `Atomic*` but I didn't feel this was necessary given the expected use cases. I am open to feedback though. Lastly, please note that although the PR looks big, 1300 lines are just tests 🥲 -- 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...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
