[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-22 Thread vanzin
Github user vanzin commented on the issue:

https://github.com/apache/spark/pull/20867
  
I don't know, it didn't use to be possible to change the base branch. If it 
is now, just change it to master.


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-22 Thread felixalbani
Github user felixalbani commented on the issue:

https://github.com/apache/spark/pull/20867
  
@vanzin I  changed the base branch from master  to branch-2.2   - 2 days 
ago - do I still need to close this and create it again? (its my first PR - my 
apologies for any inconvenience)


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/20867
  
Merged build finished. Test FAILed.


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-22 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/20867
  
**[Test build #88527 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88527/testReport)**
 for PR 20867 at commit 
[`5d7ed7a`](https://github.com/apache/spark/commit/5d7ed7ae933d2dc473abdb6f0fc0277d5cf0d7da).
 * This patch **fails Scala style tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/20867
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88527/
Test FAILed.


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-22 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/20867
  
**[Test build #88527 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88527/testReport)**
 for PR 20867 at commit 
[`5d7ed7a`](https://github.com/apache/spark/commit/5d7ed7ae933d2dc473abdb6f0fc0277d5cf0d7da).


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-22 Thread vanzin
Github user vanzin commented on the issue:

https://github.com/apache/spark/pull/20867
  
Actually, this PR is against 2.2, not master. Please close this and open a 
new one against the right branch.


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-22 Thread vanzin
Github user vanzin commented on the issue:

https://github.com/apache/spark/pull/20867
  
ok to test


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-21 Thread felixalbani
Github user felixalbani commented on the issue:

https://github.com/apache/spark/pull/20867
  
@mgaido91 I pushed the start() after setAcceptQueueSize in new commit 


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-21 Thread vanzin
Github user vanzin commented on the issue:

https://github.com/apache/spark/pull/20867
  
Yes, that's a bug (and also is being fixed as part of #20327, but ok if you 
want to fix it here).


---

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



[GitHub] spark issue #20867: [SPARK-23759][UI] Unable to bind Spark2 history server t...

2018-03-21 Thread mgaido91
Github user mgaido91 commented on the issue:

https://github.com/apache/spark/pull/20867
  
cc @vanzin 

I think this comes from SPARK-17874. I see that some properties (the 
connector name in particular) are set after this method. So I guess they have 
no effect too. Shall we move them here before the start too?


---

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



[GitHub] spark issue #20867: Spark 23759

2018-03-20 Thread wangyum
Github user wangyum commented on the issue:

https://github.com/apache/spark/pull/20867
  
Please update the title.


---

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



[GitHub] spark issue #20867: Spark 23759

2018-03-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/20867
  
Can one of the admins verify this patch?


---

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



[GitHub] spark issue #20867: Spark 23759

2018-03-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/20867
  
Can one of the admins verify this patch?


---

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