Elliotte Rusty Harold
Elliotte Rusty Harold
Ping. I just hit this again. It's noise. It should go.
It's not just one test doing this but these might be a couple: ``` 2024-07-17T16:42:08.484-0600 INFO -- Loaded temp storage local -- 2024-07-17T16:42:08.501-0600 WARNING Tests from com.facebook.presto.operator.aggregation.TestApproximateCountDistinctIpAddress took 1.28m 2024-07-17T16:42:08.655-0600...
I think in all cases here each method still tests what it tested before. Only setup and verification code has been rewritten to use non-deprecated methods and to be more...
That would be nice. Unfortunately, "An implementation may override the default charset with the system property file.encoding on the command line. If the value is COMPAT, the default charset is...
I need to take another look at this, but I'm not convinced the existing tests are correct or testing what they think they're testing.
Might be flaky or real, not sure yet: ``` Error: Tests run: 8323, Failures: 7, Errors: 0, Skipped: 1, Time elapsed: 1,360.423 s
If you think we need to keep these bindings, there are two questions that need to be answered: 1. What test is missing? There are lot of integration tests in...
I made some updates to the Wiki. Seems it is asking folks to do things that can't yet be done.
Question for @cstamas what observable effect do you expect from merging this PR? As far as I can tell the only change is that the jar file is a little...
Leaving open for useful context