jdesjean commented on PR #41443: URL: https://github.com/apache/spark/pull/41443#issuecomment-1604796450
> > @beliefer the event API is open by design, it is made for others to use it. For example thriftserver uses it. This particular PR will be foundation on which we will be building a UI for connect. > > Thank you for the explanation. Can existing event types be shared by thriftserver and connect? The Thrift events are flexible and semantically the same as the Connect events. It's technically possible to reuse them, however there are some differences between both implementations. I believe it's preferable to create a new set of events so that both projects (Thrift & Connect) can evolve separately. -- 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
