andygrove commented on a change in pull request #10026:
URL: https://github.com/apache/arrow/pull/10026#discussion_r614092643



##########
File path: rust/ballista/ui/scheduler/README.md
##########
@@ -0,0 +1,26 @@
+# Ballista UI
+
+
+## Available Scripts
+
+In the project directory, you can run:
+
+### `yarn start`

Review comment:
       I'm curious ... why Yarn? 
   
   If we want to support yarn then we should add some documentation here such 
as "What is Yarn" and "How to install Yarn" but I am concerned that adding Yarn 
here implies we need to support Ballista on Yarn as well.
   
   What would the alternatives be here for local testing? Cargo run? Docker 
compose?
   
   




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

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


Reply via email to