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

Reply via email to