timothy65535 edited a comment on pull request #1023: URL: https://github.com/apache/incubator-kyuubi/pull/1023#issuecomment-914169452
hi @yaooqinn, looking forward to your suggestions Had read the whole ui design of sparkthriftserver https://github.com/apache/spark/tree/master/sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/ui And already tried toiImplement a memory status store https://github.com/timothy65535/kyuubi/blob/ky-981-t1/externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/spark/kyuubi/ui/EngineStatusStore.scala At last, in terms of design compatibility and simplicity, I think it's better to reuse the thriftserver's ui design. We need to consider not only the pages on engine server, but also the pages on spark history server. -- 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]
