[GitHub] [incubator-iceberg] rdblue commented on issue #494: TestScanSummary appears to be flaky

2019-09-30 Thread GitBox
rdblue commented on issue #494: TestScanSummary appears to be flaky URL: https://github.com/apache/incubator-iceberg/issues/494#issuecomment-536688978 Yeah, we can update that logic to be a bit more paranoid about values we get from System.currentTimeMillis instead of using equality.

[GitHub] [incubator-iceberg] rdblue commented on issue #494: TestScanSummary appears to be flaky

2019-09-30 Thread GitBox
rdblue commented on issue #494: TestScanSummary appears to be flaky URL: https://github.com/apache/incubator-iceberg/issues/494#issuecomment-536659666 Which test case was the problem? Was it the same, or did I miss one? This

[GitHub] [incubator-iceberg] rdblue commented on issue #494: TestScanSummary appears to be flaky

2019-09-25 Thread GitBox
rdblue commented on issue #494: TestScanSummary appears to be flaky URL: https://github.com/apache/incubator-iceberg/issues/494#issuecomment-535111262 Which test cases were flaky? We added a fix for this a little while ago: