milenkovicm commented on PR #1670: URL: https://github.com/apache/datafusion-ballista/pull/1670#issuecomment-4419175918
> > `CPU` cores as well > > This is a bit more involved. At the moment the Executors view uses the `/api/executors` REST API to load its data. To be able to show the CPU count I will need to make an extra HTTP request to `/api/executor/<id>` for every executor to load its extra details (the OS info). > > I see these options: > > 1. make the extra HTTP requests in the TUI code > 2. update the REST API to return all the information with `/api/executors` > 3. do nothing, and show the CPU count only in the details popup we can do it as a follow up -- 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]
