munendrasn commented on PR #6823: URL: https://github.com/apache/iceberg/pull/6823#issuecomment-1429660556
I thought of updating this [test](https://github.com/apache/iceberg/blob/ef7e20e12eae2638015ff91b3356eb2f32f601cc/aws/src/integration/java/org/apache/iceberg/aws/dynamodb/TestDynamoDbCatalog.java#L213) to create more tables, and validate the change. As DynamoDb Query reads max 1MB of data (when limit is not specified), need to create large number of tables (~10K with current setup) to reproduce the issue. Please let me know if I should go ahead, and update number of tables to higher number in the tests Locally, I could verify the fix by adding `limit` parameter to the query request and then calling listTables on DynamoDb with & w/o the fix -- 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
