Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/20134
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/20947
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/22171
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/22171
@viirya , Current issue occurs only in the case of 0 values, none zero
values with higher scale are still save in non scientific notation
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/22171
@gatorsmile @HyukjinKwon @viirya , I rechecked the customer scenario.
It seems dataframe is saved as csv file and then netezza loads the csv data
into netezza table. In csv output 0 values
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/22307
@HyukjinKwon , I'll close this PR
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/22307
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/22171
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/22307
@HyukjinKwon , even with this
```create function d100.udf100 as
'org.apache.hadoop.hive.ql.udf.generic.GenericUDFUpper'; ``` we can simulate
this issue.
I've updated PR description
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/22307
[SPARK-25301][SQL] When a view uses an UDF from a non default database,
Spark analyser throws AnalysisException
## What changes were proposed in this pull request?
When a hive view uses an UDF
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/22171#discussion_r212251852
--- Diff: sql/core/src/test/resources/sql-tests/results/literals.sql.out ---
@@ -197,7 +197,7 @@ select .e3
-- !query 20
select 1E309, -1E309
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/22171#discussion_r212243480
--- Diff: sql/core/src/test/resources/sql-tests/results/literals.sql.out ---
@@ -197,7 +197,7 @@ select .e3
-- !query 20
select 1E309, -1E309
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/22171#discussion_r212243149
--- Diff:
sql/core/src/test/resources/sql-tests/results/higher-order-functions.sql.out ---
@@ -201,6 +201,7 @@ struct<>
-- !query 20
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/22171
@viirya, This issue is not only related to Dataset.show but also in dataset
write operations. External databases like netezza fails save the result due
to scientific notation on "0&qu
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/22171
[SPARK-25177][SQL] When dataframe decimal type column having scale higher
than 6, 0 values are shown in scientific notation
## What changes were proposed in this pull request?
If scale
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/22130
@dongjoon-hyun , Thanks for taking a look at this PR, I've added Mac OS
version in the PR description,
IMO, an update of ncurses is causing this issue
Reference : https://github.com
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/22130
[SPARK-25137][Spark Shell] NumberFormatException` when starting spark-shell
from Mac terminal
## What changes were proposed in this pull request?
When starting spark-shell from Mac
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/20611#discussion_r180993462
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -370,26 +339,35 @@ case class LoadDataCommand
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/20611#discussion_r180993068
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -370,26 +339,35 @@ case class LoadDataCommand
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/20947
[SPARK-23705][SQL]Handle non-distinct columns in DataSet.groupBy
## What changes were proposed in this pull request?
If input columns to DataSet.groupBy contains non unique columns, remove
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/20917
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/20917
[SPARK-23705][SQL]Handle non-distinct columns in DataSet.groupBy
## What changes were proposed in this pull request?
If input columns to DataSet.groupBy contains non unique columns, remove
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/20134
[SPARK-22613][SQL] Make UNCACHE TABLE behaviour consistent with CACHE TABLE
## What changes were proposed in this pull request?
Added an option LAZY for UNCACHE TABLE.
Eg: UNCACHE LAZY
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/19809
Thank you.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/19809
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19809#discussion_r152922206
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -700,12 +700,7 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/19809
ping @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/19809
[SPARK-17920][SQL] [FOLLOWUP] Backport PR 19779 to branch-2.2
## What changes were proposed in this pull request?
A followup of
> https://github.com/apache/spark/pull/19
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/19795
Thank you
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/19795
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/19795
[SPARK-17920][SPARK-19580][SPARK-19878][SQL] Backport PR 19779 to
branch-2.2 - Support writing to Hive table which uses Avro schema url
'avro.schema.url'
## What changes were proposed
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/19779
@gatorsmile , @cloud-fan and @dongjoon-hyun
Thanks for the review comments and guidence
Sure, I'll submit a separate PR for backporting it to 2.2
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r152474528
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -841,6 +841,76 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r152473900
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -841,6 +841,76 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r152473845
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -841,6 +841,76 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r152464029
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -841,6 +841,75 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r152446382
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -841,6 +841,75 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r152349156
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -841,6 +841,75 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r152286208
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -800,7 +800,7 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r152067290
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/VersionsSuite.scala ---
@@ -841,6 +841,76 @@ class VersionsSuite extends SparkFunSuite
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19779#discussion_r151902033
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveFileFormat.scala
---
@@ -89,6 +90,8 @@ class HiveFileFormat(fileSinkConf
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/19779
[SPARK-17920][SPARK-19580][SPARK-19878][SQL] Support writing to Hive table
which uses Avro schema url 'avro.schema.url'
## What changes were proposed in this pull request?
Support writing
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/19008
@rxin ,
Sure, I'll update it
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19008#discussion_r134227248
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonParsingOptionsSuite.scala
---
@@ -72,6 +72,21 @@ class
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/19008#discussion_r134227247
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonParsingOptionsSuite.scala
---
@@ -72,6 +72,21 @@ class
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/19008
[SPARK-21756][SQL]Add JSON option to allow unquoted control characters
## What changes were proposed in this pull request?
This patch adds allowUnquotedControlChars option in JSON data
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/19007
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/19007
Ok , I'm closing my PR.
Now a days, Spark JIRA is not showing PR status.That is why I missed your
PR.
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/19007
[SPARK-21783][SQL]Turn on ORC filter push-down by default
## What changes were proposed in this pull request?
Turned on ORC filter push-down option by default
Github user vinodkc commented on the issue:
https://github.com/apache/spark/pull/18880
retest this
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/18880
[SPARK-21665][Core]Need to close resources after use
## What changes were proposed in this pull request?
Resources in Core - SparkSubmitArguments.scala, Spark-launcher
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/18852#discussion_r131519921
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -808,6 +808,12 @@ class SQLQuerySuite extends QueryTest
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/18852
[SPARK-21588][SQL] SQLContext.getConf(key, null) should return null
## What changes were proposed in this pull request?
In SQLContext.get(key,null) for a key that is not defined
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/18771
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/18771#discussion_r130234632
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ---
@@ -114,13 +114,13 @@ class CacheManager extends Logging
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/18771
[SPARK-21478][SQL] Avoid unpersisting related Datasets
## What changes were proposed in this pull request?
While unpersisting a dataset, only unpersist and remove that datasets's
plan from
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/8834
[SPARK-10631][Documentation, MLlib, PySpark]Added documentation for few APIs
There are some missing API docs in pyspark.mllib.linalg.Vector (including
DenseVector and SparseVector). We should add
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8682#issuecomment-140645936
Sure I'll work on SPARK-10631
Thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8730#issuecomment-139956635
Can we retest?, it is failing in unrelated module
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8730#issuecomment-139957047
jenkins retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/8730
[SPARK-10575][Spark Core]Wrapped RDD.takeSample with Scope
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vinodkc/spark fix_takesample_return
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/8233
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/8682
[SPARK-10516][ MLlib]Added values property in DenseVector
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vinodkc/spark fix_SPARK-10516
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/8507
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8507#issuecomment-138789894
Closing this PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/8636
[SPARK-10468][ MLlib ]Verify schema before Dataframe select API call
Loader.checkSchema was called to verify the schema after
dataframe.select(...).
Schema verification should be done before
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8565#issuecomment-138167561
Closing this PR, as there is already an existing JIRA & PR on same issue
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/8565
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8507#issuecomment-137686626
@feynmanliang , I've removed case classes used for schema inference.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/8507#discussion_r38733907
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/KMeansModel.scala ---
@@ -125,16 +126,19 @@ object KMeansModel extends Loader[KMeansModel
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/8565#discussion_r38507995
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/linalg/Matrices.scala
---
@@ -278,7 +278,8 @@ class DenseMatrix @Since("
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/8565
[SPARK-10414][MLlib]Fix hashcode issues in MLLib
Added/Updated hashcode methods in 3 classes
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8507#issuecomment-137312813
@feynmanliang , fixed your review comments. Could you please verify it?
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/8581
[SPARK-10430][SPARK-10430]Added hashCode methods in AccumulableInfo and
RDDOperationScope
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/8565#discussion_r38609166
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/linalg/Matrices.scala
---
@@ -278,7 +278,8 @@ class DenseMatrix @Since("
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/8565#discussion_r38522537
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/linalg/Matrices.scala
---
@@ -278,7 +278,8 @@ class DenseMatrix @Since("
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/8565#discussion_r38514784
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/linalg/Matrices.scala
---
@@ -278,7 +278,8 @@ class DenseMatrix @Since("
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/8507#discussion_r38410341
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -159,19 +159,25 @@ object GaussianMixtureModel extends
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/8507
[SPARK-10199][MLlib]Avoid using reflections for parquet model save
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vinodkc/spark fix_SPARK
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/8233#issuecomment-134471497
Sure, I'll update the code
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/8233
[SPARK-10016][ML]Stored broadcast var in a private var
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vinodkc/spark fix_SPARK-10016
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/7325#issuecomment-125820807
@mengxr , I've added message in JIRA page from my JIRA account.
Thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/7364
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/7364#issuecomment-121303647
Closing this incomplete PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user vinodkc closed the pull request at:
https://github.com/apache/spark/pull/7040
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/7364#discussion_r34534208
--- Diff: core/src/main/scala/org/apache/spark/deploy/master/Master.scala
---
@@ -727,40 +727,44 @@ private[master] class Master(
def
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/7367
[SPARK-8991][ML]Update SharedParamsCodeGen's Generated Documentation
Removed private[ml] from Generated documentation
You can merge this pull request into a Git repository by running:
$ git
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/7364
[SPARK-8761][Deploy]Added synchronization in removeApplication
Master.removeApplication is not thread-safe. But it's called both in the
message loop of Master and MasterPage.handleAppKillRequest
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/7325
[SPARK-8919][Documentation, MLlib]Added @since tags to mllib.recommendation
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vinodkc/spark
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/7040#discussion_r34328058
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/conditionals.scala
---
@@ -296,9 +295,7 @@ case class CaseKeyWhen(key
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/7183
[SPARK-8787][SQL]Changed parameter order of @deprecated in package object
sql
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vinodkc/spark
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/7183#discussion_r33763973
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/package.scala ---
@@ -46,6 +46,6 @@ package object sql {
* Type alias for [[DataFrame]]. Kept
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/7183#issuecomment-117957848
Added description
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user vinodkc commented on a diff in the pull request:
https://github.com/apache/spark/pull/7015#discussion_r33558094
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/AbstractSparkSQLParser.scala
---
@@ -30,12 +30,14 @@ private[sql] abstract class
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/7015#issuecomment-115572040
Python test failure in unrelated module mllib
File
/home/jenkins/workspace/SparkPullRequestBuilder/python/pyspark/mllib/classification.py,
line 106
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/7040#issuecomment-115654691
Ok, I'll update that test suite.
@smola , In predicates.scala, eval method of case class EqualNullSafe has
similar if (l == null r == null) check , shall I
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/7040
[SPARK-8636][SQL] Fix equalNullSafe comparison
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vinodkc/spark fix_CaseKeyWhen_equalNullSafe
GitHub user vinodkc opened a pull request:
https://github.com/apache/spark/pull/7015
[SPARK-8628][SQL]Race condition in AbstractSparkSQLParser.parse
Added synchronization in 'initialize' method
You can merge this pull request into a Git repository by running:
$ git pull https
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/7015#issuecomment-115492255
@smola ,I've updated the code.Can you please check whether that fix solves
the issue ?
---
If your project is set up for it, you can reply to this email and have your
Github user vinodkc commented on the pull request:
https://github.com/apache/spark/pull/5997#issuecomment-100585781
@marmbrus , PR https://github.com/apache/spark/pull/6013 handled it
---
If your project is set up for it, you can reply to this email and have your
reply appear
1 - 100 of 130 matches
Mail list logo