[GitHub] [spark] Yikun edited a comment on pull request #35663: [SPARK-37809][K8S] Add yunikorn feature step

[GitBox <gi...@apache.org>]

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 annotations placeholder support which @dongjoon-hyun  introduced. In theory, all annotations can be set by this way.(such as queue placeholder). It's very flexiable. This is very helpful for custom schdulers which only need annotaions 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 CRD and also need many annotaions.
   
   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, `Yunikorn` can also doc in spark website, tell user how to use offcially.


-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org