[GitHub] [spark] holdenk commented on pull request #32436: [SPARK-34764][CORE][K8S][UI] Propagate reason for exec loss to Web UI

2021-05-04 Thread GitBox
holdenk commented on pull request #32436: URL: https://github.com/apache/spark/pull/32436#issuecomment-832158404 cc @dongjoon-hyun & @BryanCutler -- 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

[GitHub] [spark] holdenk commented on pull request #32436: [SPARK-34764][CORE][K8S][UI] Propagate reason for exec loss to Web UI

2021-05-05 Thread GitBox
holdenk commented on pull request #32436: URL: https://github.com/apache/spark/pull/32436#issuecomment-832841187 So if you want it's pretty easy to trigger OOMs with the GroupByKey example (it's how I did the manual test). I'm not sure about event logs though, do we have those committed so

[GitHub] [spark] holdenk commented on pull request #32436: [SPARK-34764][CORE][K8S][UI] Propagate reason for exec loss to Web UI

2021-05-11 Thread GitBox
holdenk commented on pull request #32436: URL: https://github.com/apache/spark/pull/32436#issuecomment-839237499 K8s failure is unrelated (`The connection to the server localhost:8080 was refused - did you specify the right host or port?`). I'll merge this Thursday unless anyone has additi

[GitHub] [spark] holdenk commented on pull request #32436: [SPARK-34764][CORE][K8S][UI] Propagate reason for exec loss to Web UI

2021-05-13 Thread GitBox
holdenk commented on pull request #32436: URL: https://github.com/apache/spark/pull/32436#issuecomment-840881048 Going to merge now, thanks everyone for the review and feedback. We can iterate on adding more exit codes in future PRs if we see any common questions on the user@ list :) --

[GitHub] [spark] holdenk commented on pull request #32436: [SPARK-34764][CORE][K8S][UI] Propagate reason for exec loss to Web UI

2021-05-13 Thread GitBox
holdenk commented on pull request #32436: URL: https://github.com/apache/spark/pull/32436#issuecomment-840883315 Merged to the current dev branch :) Since it's a new feature not planning on back porting. -- This is an automated message from the Apache Git Service. To respond to the messa