[GitHub] [hadoop] steveloughran commented on issue #1766: HADOOP-16767 Handle non-IO exceptions in reopen()

2020-03-02 Thread GitBox
steveloughran commented on issue #1766: HADOOP-16767 Handle non-IO exceptions in reopen() URL: https://github.com/apache/hadoop/pull/1766#issuecomment-593514135 well, its a minor patch so I'll D/L and test myself, ` -Dparallel-tests -DtestsThreadCount=8 -Ds3guard -Ddynamo` So +1, me

[GitHub] [hadoop] steveloughran commented on issue #1766: HADOOP-16767 Handle non-IO exceptions in reopen()

2020-01-21 Thread GitBox
steveloughran commented on issue #1766: HADOOP-16767 Handle non-IO exceptions in reopen() URL: https://github.com/apache/hadoop/pull/1766#issuecomment-576716575 Still expecting that declaration of which s3 endpoint you ran the `mvn verify` tests for hadoop-aws. we don't have time to run ev

[GitHub] [hadoop] steveloughran commented on issue #1766: HADOOP-16767 Handle non-IO exceptions in reopen()

2020-01-02 Thread GitBox
steveloughran commented on issue #1766: HADOOP-16767 Handle non-IO exceptions in reopen() URL: https://github.com/apache/hadoop/pull/1766#issuecomment-570237105 makes sense, patch LGTM and I don't see how you'd easily write a test for it. one little bit of homework: were did you run