KeeProMise commented on PR #6622: URL: https://github.com/apache/hadoop/pull/6622#issuecomment-2027895211
> Was gonna hit the merge button, but got curious. > > Earlier if the value was 1, was it an infinite loop always? or in just in a corner case, if in just one corner case & now if we throw an exception, if anyone has value of 1 earlier his jobs would be working but post this change it would fail? > > can we just break the condition for infinite loop if that is the case & preserve the value? or at worst if that is an illegal value, we put a warn log & use the defaul? > > @brumi1024 / @szilard-nemeth any pointers? Hi. The code related to this method has been running for a very long time (since June 13, 2011), and I suspect that setting this value to 1 is meaningless in the actual generation environment. Therefore, this bug should have existed for a long time, but has not been reported yet. Therefore, I believe that if the value is 1, throwing an exception will not have any impact on the existing business. -- 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