Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r197449373
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,187 @@
+/*
+ *
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r197444068
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,187 @@
+/*
+ *
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21068
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r196186937
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licen
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r196087949
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,10 @@ package object config {
CACHE
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195922985
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,187 @@
+/*
+ *
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195896675
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,187 @@
+/*
+ *
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195896638
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,187 @@
+/*
+ *
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195750196
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,187 @@
+/*
+ *
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195745591
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,187 @@
+/*
+ * Licen
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195566345
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -151,9 +149,9 @@ private[spark] clas
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195476043
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,189 @@
+/*
+ *
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195467887
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,189 @@
+/*
+ * Licen
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r195469104
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,189 @@
+/*
+ * Licen
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r191762829
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,19 @@ package object config {
CAC
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r191520704
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,19 @@ package object config {
CACHE
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185791498
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185619109
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureTracker.scala
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed to the Apach
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185570861
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,19 @@ package object config {
CACHED_F
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r18979
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
+ *
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185534467
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorSuite.scala
---
@@ -354,7 +358,7 @@ class YarnAllocatorSuite exten
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185528716
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -273,7 +273,7 @@ private[scheduler] class BlacklistTracker (
*
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185556534
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
+ *
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185529548
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureTracker.scala
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed to the Apach
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185556226
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
+ *
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185539768
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -602,8 +569,7 @@ private[yarn] class YarnAllocator(
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185549149
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
+ *
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185558661
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
+ *
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185543512
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,145 @@
+/*
+ * Licen
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185532512
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,145 @@
+/*
+ * Licen
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185543919
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,145 @@
+/*
+ * Licen
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185545685
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,144 @@
+/*
+ *
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185533750
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -102,18 +102,14 @@ private[yarn] class YarnAllocator
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185447393
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
---
@@ -170,8 +170,7 @@ class CoarseGrainedSchedul
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185441810
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ *
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185440657
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ *
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185435494
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureTracker.scala
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed to the
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185434976
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureTracker.scala
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed to the
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185432837
--- Diff: core/src/test/scala/org/apache/spark/HeartbeatReceiverSuite.scala
---
@@ -73,6 +73,7 @@ class HeartbeatReceiverSuite
sc = spy(new Spa
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185111225
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,18 @@ package object config {
CACHED_F
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185106176
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
---
@@ -170,8 +170,7 @@ class CoarseGrainedSchedulerBac
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185110988
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185110389
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185106301
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureTracker.scala
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed to the Apach
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185105266
--- Diff: core/src/test/scala/org/apache/spark/HeartbeatReceiverSuite.scala
---
@@ -73,6 +73,7 @@ class HeartbeatReceiverSuite
sc = spy(new SparkCon
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185110457
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r18531
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185106409
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureTracker.scala
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed to the Apach
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185113461
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185112100
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185109185
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureTracker.scala
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed to the Apach
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185109769
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureTracker.scala
---
@@ -0,0 +1,85 @@
+/*
+ * Licensed to the Apach
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185110730
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185110240
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -204,35 +182,23 @@ private[yarn] class YarnAllocator
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184575258
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
---
@@ -170,8 +170,7 @@ class CoarseGrainedSchedulerBac
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184218440
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184218320
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184211525
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -651,8 +651,8 @@ private[spark] class TaskSchedulerImpl(
* Ge
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184217319
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184220567
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,155 @@
+/*
+ *
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184212508
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,79 @@
+/*
+ * Lic
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184211931
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
---
@@ -170,8 +170,8 @@ class CoarseGrainedSchedulerBac
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184214954
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,79 @@
+/*
+ * Lic
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184219520
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184217451
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184215254
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,79 @@
+/*
+ * Lic
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184220488
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTrackerSuite.scala
---
@@ -0,0 +1,155 @@
+/*
+ *
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184212576
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,79 @@
+/*
+ * Lic
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184216378
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -204,35 +185,23 @@ private[yarn] class YarnAllocator
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184217648
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184216617
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -204,35 +185,23 @@ private[yarn] class YarnAllocator
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184217726
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184219340
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184217554
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184219746
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,19 @@ package object config {
CACHED_F
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184210860
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -126,7 +126,7 @@ private[scheduler] class BlacklistTracker (
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184219685
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licen
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r184215914
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala
---
@@ -160,26 +155,12 @@ private[yarn] class YarnAllocator
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182448168
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,26 @@ package object config {
CAC
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182447935
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,26 @@ package object config {
CAC
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182428059
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -216,6 +216,10 @@ private[scheduler] class BlacklistTracker (
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182427448
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,26 @@ package object config {
CAC
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182419122
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,80 @@
+/*
+
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182418247
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,80 @@
+/*
+
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182411922
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,80 @@
+/*
+
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182407936
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,155 @@
+/*
+ *
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182405470
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,155 @@
+/*
+ *
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r182404677
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,155 @@
+/*
+ *
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181880503
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,26 @@ package object config {
CACHE
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181881863
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,26 @@ package object config {
CACHE
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181814472
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/BlacklistTracker.scala ---
@@ -216,6 +216,10 @@ private[scheduler] class BlacklistTracker (
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181882445
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/config.scala
---
@@ -328,4 +328,26 @@ package object config {
CACHE
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181908529
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,155 @@
+/*
+ * Licen
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181910914
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Lic
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181907316
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,155 @@
+/*
+ * Licen
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181912300
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Lic
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181907395
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,155 @@
+/*
+ * Licen
Github user henryr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181911744
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Lic
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181513236
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/FailureWithinTimeIntervalTracker.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Lic
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r181515465
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala
---
@@ -0,0 +1,155 @@
+/*
+ * Licen
1 - 100 of 101 matches
Mail list logo