AngersZhuuuu commented on pull request #31522: URL: https://github.com/apache/spark/pull/31522#issuecomment-794964106
> We can periodically log something in the built-in file commit protocol, but there is nothing we can do if people are using a custom file commit protocol. A new thread to log these can be ok, but looks weird. (We use this way to show thrift server's progress) > I checked other plan nodes and found that the file scan node has a "metadata time" metrics. I think it makes sense to have something similar in the write nodes, but we need to think about the naming and what to include (shall we include the hive LOAD TABLE time?). If possible, I think the more comprehensive information the better. As I mentioned in https://github.com/apache/spark/pull/31522#issuecomment-793672126, if we have `LOAD TABLE` time in SQL Tab's node, it will be easier for us to explain to our user/customer. ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org