Yikun edited a comment on pull request #35663:
URL: https://github.com/apache/spark/pull/35663#issuecomment-1056101229


   @yangwwei 
   
   There are two alternative way to support `Yunikorn` at this stage:
   
   - New annotation placeholder support which @dongjoon-hyun  introduced. In 
theory, all annotations can be set this way.(such as queue placeholder). It's 
very flexible. This is very helpful for custom schedulers which only need 
annotations set.
   
   - A separate feature step (this PR): this will help to simplify annotation 
configuration, we just need to set feature step to instead of setting many 
annotations conf. This way it's more friendly to schedulers which need to 
create extra CRD and also need many annotations.
   
   I think dongjoon-hyun's concern is how to use the most simple way to 
integrate Yunikorn at this stage. And I also think even if we support by this 
way, spark with `Yunikorn` can also doc in spark website, tell user how to use 
officially.


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