You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@yunikorn.apache.org by GitBox <gi...@apache.org> on 2022/10/16 07:26:04 UTC

[GitHub] [yunikorn-site] 0yukali0 commented on a diff in pull request #197: [YUNIKORN-1358] Change queue name examples for submitting spark jobs in doc.

0yukali0 commented on code in PR #197:
URL: https://github.com/apache/yunikorn-site/pull/197#discussion_r996402127


##########
docs/user_guide/workloads/run_spark.md:
##########
@@ -131,8 +131,8 @@ Assigning the applicationId and the queue path are possible.
 ```
 --conf spark.kubernetes.executor.label.applicationId=application-spark-0001
 --conf spark.kubernetes.driver.label.applicationId=application-spark-0001
---conf spark.kubernetes.executor.label.queue=default.root.sandbox
---conf spark.kubernetes.driver.label.queue=default.root.sandbox
+--conf spark.kubernetes.executor.label.queue=root.default.sandbox

Review Comment:
   Modification:
   This queue path could be root.default.sandbox -> root.sandbox
   
   Idea:
   This same issue also appears in "i18n/zh-cn/docusaurus-plugin-content-docs/current/user_guide/workloads/run_spark.md".
   Could you modify that file?



-- 
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@yunikorn.apache.org

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