dongjoon-hyun commented on PR #53231:
URL: https://github.com/apache/spark/pull/53231#issuecomment-3587584104

   However, we maintained both setter and getter methods like Apache Hadoop 
`AuxiliaryService` abstract class. I'm not sure it's a good idea for us to 
remove the only way to get the recovery path at Apache Spark 4.2.0.
   ```java
   public void setRecoveryPath(Path recoveryPath) {
   protected Path getRecoveryPath(String fileName) {
   ```
   
   IIUC, this is public class, isn't it?
   
   
https://github.com/apache/spark/blob/154a2708e98840ba4d69b5afdd549e695081e4ce/common/network-yarn/src/main/java/org/apache/spark/network/yarn/YarnShuffleService.java#L101


-- 
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]

Reply via email to