HeartSaVioR commented on PR #54550: URL: https://github.com/apache/spark/pull/54550#issuecomment-3979073964
@eason-yuchen-liu Could you please update the PR title and description to describe the new change in this PR instead of just duplicating the old one? Also it doesn't look like any existing test wasn't failing and we don't add any test here. This looks like just to clarify the time unit for the parameter - that's already actually mentioned in the method doc. I'm OK with further clarification rather than relying on method doc; just want to understand the actual change here. -- 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]
