dongjoon-hyun commented on issue #23958: [SPARK-27045] SQL tab in UI shows actual SQL instead of callsite URL: https://github.com/apache/spark/pull/23958#issuecomment-471057426 @srowen . Yes. I agree that the default should be `0`. > I wouldn't argue strongly against a flag, if the default is 'no truncation' @cloud-fan . In general, for long callSite performance issues, Spark used `withDummyCallSite` as a workaround. However, there is no configuration for event logs, isn't it? BTW, this PR is trying to replace `short callSite` to somewhat unpredictable longer SQL statement. Usually, BI queries are longer than expected. That's the reason we have a longer discussion in this PR. @ajithme I agree with the following of course. For me, `config2` is a must-have item. For `config1`, please follow other committers' opinion. I'm neutral. (It's good to ignore) > I think SQL tab primary responsibility should be showing the Entire SQL of what user is trying to run.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
