[GitHub] [hadoop-ozone] sodonnel commented on issue #678: HDDS-3179 Pipeline placement based on Topology does not have fallback

2020-03-27 Thread GitBox
sodonnel commented on issue #678: HDDS-3179 Pipeline placement based on 
Topology does not have fallback
URL: https://github.com/apache/hadoop-ozone/pull/678#issuecomment-604854734
 
 
   All tests are green except this one, which I have seen fail in several other 
PRs, so it is flaky:
   
   ```
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 
52.235 s - in 
org.apache.hadoop.ozone.freon.TestDataValidateWithUnsafeByteOperations
   [INFO] Running org.apache.hadoop.ozone.freon.TestRandomKeyGenerator
   [ERROR] Tests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 
338.009 s <<< FAILURE! - in org.apache.hadoop.ozone.freon.TestRandomKeyGenerator
   [ERROR] bigFileThan2GB(org.apache.hadoop.ozone.freon.TestRandomKeyGenerator) 
 Time elapsed: 276.915 s  <<< FAILURE!
   java.lang.AssertionError: expected:<1> but was:<0>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:743)
at org.junit.Assert.assertEquals(Assert.java:118)
   ```


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 comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org



[GitHub] [hadoop-ozone] sodonnel commented on issue #678: HDDS-3179 Pipeline placement based on Topology does not have fallback

2020-03-26 Thread GitBox
sodonnel commented on issue #678: HDDS-3179 Pipeline placement based on 
Topology does not have fallback
URL: https://github.com/apache/hadoop-ozone/pull/678#issuecomment-604486699
 
 
   Thanks for quickly addressing the final issues. 
   
   I am +1 on this now. I will commit it later pending the CI checks looking 
good.


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 comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org



[GitHub] [hadoop-ozone] sodonnel commented on issue #678: HDDS-3179 Pipeline placement based on Topology does not have fallback

2020-03-26 Thread GitBox
sodonnel commented on issue #678: HDDS-3179 Pipeline placement based on 
Topology does not have fallback
URL: https://github.com/apache/hadoop-ozone/pull/678#issuecomment-604452924
 
 
   Thanks for the updates here. I think the code looks much cleaner now with 
the debug statements and refactored block in getResultSet().
   
   There are just a couple of minor changes needed to finish this one off.


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 comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org



[GitHub] [hadoop-ozone] sodonnel commented on issue #678: HDDS-3179 Pipeline placement based on Topology does not have fallback

2020-03-26 Thread GitBox
sodonnel commented on issue #678: HDDS-3179 Pipeline placement based on 
Topology does not have fallback
URL: https://github.com/apache/hadoop-ozone/pull/678#issuecomment-604425486
 
 
   Acceptance is failing with:
   
   ```

==
   Execute PI calculation| FAIL 
|
   1 != 0
   
--
   Execute WordCount | FAIL 
|
   1 != 0
   
--
   ozonesecure-mr-mapreduce :: Execute MR jobs   | FAIL 
|
   ```
   
   This is known issue and will be fixed soon - HDDS-3284.
   
   Integration tests are flaky. it-Freon:
   
   ```
   [ERROR] Tests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 
395.168 s <<< FAILURE! - in org.apache.hadoop.ozone.freon.TestRandomKeyGenerator
   [ERROR] bigFileThan2GB(org.apache.hadoop.ozone.freon.TestRandomKeyGenerator) 
 Time elapsed: 326.297 s  <<< FAILURE!
   java.lang.AssertionError: expected:<1> but was:<0>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:743)
at org.junit.Assert.assertEquals(Assert.java:118)
   ```
   
   One of these issues covers it:  
https://issues.apache.org/jira/browse/HDDS-3266 and it-freon: 
https://issues.apache.org/jira/browse/HDDS-3257
   
   it-client I am not sure


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 comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org