[GitHub] [hadoop] virajjasani commented on pull request #4552: HADOOP-18832. remove rs-api dependency (3.3 branch)

2022-07-14 Thread GitBox


virajjasani commented on PR #4552:
URL: https://github.com/apache/hadoop/pull/4552#issuecomment-1184756374

   Except for `TestRouterRpc` and `TestRouterRpcMultiDestination`, other tests 
are not consistent failures. Shall we have a couple more rounds of builds 
@ayushtkn @pjfanning, or you think these results are 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.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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



[GitHub] [hadoop] virajjasani commented on pull request #4552: HADOOP-18832. remove rs-api dependency (3.3 branch)

2022-07-13 Thread GitBox


virajjasani commented on PR #4552:
URL: https://github.com/apache/hadoop/pull/4552#issuecomment-1183921592

   The changes look good, basically this patch is (revert of HADOOP-18033 + 
Jackson 2.12.7), QA looks good and since Tez tests are now passing as per 
@ayushtkn's testing, it seems this is good enough to unblock 3.3.4. +1 
(non-binding) from my side.

   Thank you @pjfanning @ayushtkn !!


-- 
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: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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