[GitHub] [hadoop] virajjasani commented on pull request #4552: HADOOP-18832. remove rs-api dependency (3.3 branch)
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)
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