Github user superbobry commented on a diff in the pull request:
https://github.com/apache/spark/pull/19992#discussion_r159153806
--- Diff: core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala
---
@@ -2022,12 +1947,7 @@ private[spark] object JsonProtocolSuite extends
Github user chetkhatri commented on a diff in the pull request:
https://github.com/apache/spark/pull/20070#discussion_r159152519
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LatentDirichletAllocationExample.scala
---
@@ -46,7 +46,10 @@ object
Github user sharkdtu commented on the issue:
https://github.com/apache/spark/pull/20078
@felixcheung
At the beginning, if numReceivers > totleExecutorCores, there is not cpu
cores for batch processing, and `ExecutorAllocationManager` can't listen
metrics of any batches. As a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20129
**[Test build #85577 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85577/testReport)**
for PR 20129 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20133
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20133
**[Test build #85576 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85576/testReport)**
for PR 20133 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20133
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85576/
Test FAILed.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20132
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20132
The simplified logic for encoder looks good to me.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20127#discussion_r159150900
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -95,6 +98,17 @@ class Analyzer(
this(catalog,
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20127#discussion_r159150948
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -70,6 +71,8 @@ object AnalysisContext {
}
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/20124
(Happy new year!) Just commented on the JIRA; let me know what you think.
---
-
To unsubscribe, e-mail:
101 - 112 of 112 matches
Mail list logo
