Re: [PR] fix: avoid to create operator of memory storage every time [iceberg-rust]

2024-09-26 Thread via GitHub
liurenjie1024 merged PR #635: URL: https://github.com/apache/iceberg-rust/pull/635 -- 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] fix: avoid to create operator of memory storage every time [iceberg-rust]

2024-09-26 Thread via GitHub
liurenjie1024 commented on PR #635: URL: https://github.com/apache/iceberg-rust/pull/635#issuecomment-2376490188 Thanks @ZENOTME for this. -- 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 spec

[PR] fix: avoid to create operator of memory storage every time [iceberg-rust]

2024-09-18 Thread via GitHub
ZENOTME opened a new pull request, #635: URL: https://github.com/apache/iceberg-rust/pull/635 The following assert will fail because the memory storage will create a new operator every time. To maintain the context, we should create the operator at init and clone it later. ``` #[t