HeartSaVioR commented on issue #23260: [SPARK-26311][CORE] New feature: apply custom log URL pattern for executor log URLs in SHS URL: https://github.com/apache/spark/pull/23260#issuecomment-457021478 I reread the comment and realized it can be simply resolved when you construct your custom log URL to just point to the URL which serves log file listing. Once `{{FILE_NAME}}` pattern is not given in custom log URL, it doesn't deal with `{{LOG_FILES}}` and serve the URL as "log". https://github.com/apache/spark/blob/944b30a127b7855b7f59b7f743c351d14bc5d07a/core/src/main/scala/org/apache/spark/deploy/history/HistoryAppStatusStore.scala#L102-L109
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
