Hean-Chhinling commented on PR #8222: URL: https://github.com/apache/hadoop/pull/8222#issuecomment-3932747739
The parametrised unit-test`testUpdateReservation` fail with status code 400 instead of 200. This is because the client request is invalid. Maybe because of this JSON file [here](https://github.com/apache/hadoop/blob/170115ae4251fd537ffd95047923d47e54dd4552/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesReservation.java#L1099) ``` [ERROR] Tests run: 160, Failures: 4, Errors: 0, Skipped: 0, Time elapsed: 898.5 s <<< FAILURE! -- in org.apache.hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesReservation [ERROR] org.apache.hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesReservation.testUpdateReservation(int, boolean)[2] -- Time elapsed: 4.373 s <<< FAILURE! org.opentest4j.AssertionFailedError: Expecting: <200> to be equal to: <400> but was not. at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77) at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500) at org.apache.hadoop.yarn.webapp.WebServicesTestUtils.assertResponseStatusCode(WebServicesTestUtils.java:136) at org.apache.hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesReservation.updateReservationTestHelper(TestRMWebServicesReservation.java:1127) at org.apache.hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesReservation.testUpdateReservation(TestRMWebServicesReservation.java:434) ``` Then I tested running the unit-test without this PR changes. It still fails with status code 400 for path `reservation/update` Thus these unit-tests failure at `testUpdateReservation` is not because of changes in this PR. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
