tgravescs commented on pull request #33135: URL: https://github.com/apache/spark/pull/33135#issuecomment-870847874
I'm pretty hesitant about this without more information. I do not want yet another plugin place that we have to maintain and is a one off use. These are all internal Spark classes as well, so making a config for it without some public or developer interface doesn't make a lot of sense. cc @mridulm as well Can you give more details about exactly what requires changing here? > there may be significant (non-generic) changes required in Yarn client If its a different yarn Client, if its way different it should just be a separate resource manager perhaps. If its something smaller (which it doesn't sound like from the description) perhaps something else needs to be more flexible. -- 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]
