[GitHub] spark issue #20640: [SPARK-19755][Mesos] Blacklist is always active for Meso...

2018-03-27 Thread hantuzun
Github user hantuzun commented on the issue:

https://github.com/apache/spark/pull/20640
  
Thanks a lot for the work here.

Wouldn't it be better if we change the log in the last commit 
(https://github.com/apache/spark/pull/20640/commits/66ed5afae1a5b4856e84c805c7858d552f38b26a)
 like below? 
```diff
- Mesos slave $slaveId failed
+ Task $taskId failed on Mesos slave $slaveId.
```

I'm not sure about logging convention of the project but I feel like 
logging it as `error` could be an option.

---

OOT: Having Spark 2.3 released, how could we have 
https://spark.apache.org/versioning-policy.html updated with Spark 2.4 release 
window?


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #17619: [SPARK-19755][Mesos] Blacklist is always active for Meso...

2018-02-15 Thread hantuzun
Github user hantuzun commented on the issue:

https://github.com/apache/spark/pull/17619
  
Even though we only run normal Spark jobs this PR is going to fix a case 
for us as well.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org