Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2024-09-11 Thread via GitHub
github-actions[bot] closed issue #7154: Replace Thread.sleep() usage in test code with Awaitility URL: https://github.com/apache/iceberg/issues/7154 -- 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 t

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2024-09-11 Thread via GitHub
github-actions[bot] commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-2345009924 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2024-08-27 Thread via GitHub
github-actions[bot] commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-2313786070 This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occurs.

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2023-10-30 Thread via GitHub
nastra commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-1785003366 I think a few other places that would be good to address are all places that use https://github.com/apache/iceberg/blob/main/spark/v3.5/spark-extensions/src/test/java/org/apache/icebe

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2023-10-30 Thread via GitHub
nastra closed issue #7154: Replace Thread.sleep() usage in test code with Awaitility URL: https://github.com/apache/iceberg/issues/7154 -- 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 c

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2023-10-28 Thread via GitHub
coded9 commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-1783824507 It seems all usages were replaced. Are we good to close this @nastra ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2023-10-17 Thread via GitHub
nastra commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-1766792091 I've opened https://github.com/apache/iceberg/pull/8853 and https://github.com/apache/iceberg/pull/8852 to give an idea about places that are good candidates to replace with Awaitilit

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2023-10-06 Thread via GitHub
shreyanshR7 commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-1750854267 @nastra please consider the pr #8725 -- 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 g

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2023-10-04 Thread via GitHub
shreyanshR7 commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-1747275843 @nastra I tried to implement the above method. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2023-10-03 Thread via GitHub
gzagarwal commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-1745318517 okay let me find something else. -- 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

Re: [I] Replace Thread.sleep() usage in test code with Awaitility [iceberg]

2023-10-02 Thread via GitHub
nastra commented on issue #7154: URL: https://github.com/apache/iceberg/issues/7154#issuecomment-1742597094 @gzagarwal you might want to talk to @coded9 as he also wanted to work on this, although I don't know the current state of this -- This is an automated message from the Apache Git S