Thanks a lot Vladimir, your fix on master indeed solved my issue.

Unfortunately this time AppVeyor failed on ElasticSearchAdapterTest due to
a "java.net.SocketTimeoutException" (link to the build
https://ci.appveyor.com/project/ApacheSoftwareFoundation/calcite/builds/30510892/job/0o5meoj2omuxadi8
).

What do you usually do in such cases (transient failures)?

Shall I wait or try to re-trigger Travis again?

Best regards,
Alessandro

On Sat, 1 Feb 2020 at 22:50, Vladimir Sitnikov <[email protected]>
wrote:

> >My PR is this one: https://github.com/apache/calcite/pull/1774
>
> Ok. I see. The error in your PR is exactly the same.
> I guess it was caused by
>
> https://github.com/apache/calcite/commit/bcac62e3dad6137511d4451135daa2d1762ec6ad#diff-c25007f5834d66d28e0b6644edf21325L55-R69
>
> In other words, `master` branch build was broken, and PR is validated by
> merging with master branch.
> I've fixed master branch, so CI in your PR would likely work.
>
> Vladimir
>

Reply via email to