Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23132
> mind fixing PR description accordingly?
@HyukjinKwon fixed
---
-
To unsubscribe, e-mail: reviews-unsub
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r240041107
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -347,17 +347,28 @@ class JacksonParser
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23201#discussion_r240038837
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonInferSchema.scala
---
@@ -121,7 +122,26 @@ private[sql] class JsonInferSchema
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23201#discussion_r240031238
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonInferSchema.scala
---
@@ -121,7 +122,26 @@ private[sql] class JsonInferSchema
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r240030751
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/TestJsonData.scala
---
@@ -229,6 +229,11 @@ private[json] trait
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r240029821
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -35,7 +35,9 @@ displayTitle: Spark SQL Upgrading Guide
- Since Spark 3.0, CSV datasource
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23257
@cloud-fan What do you think of the PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r240014440
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -37,6 +37,8 @@ displayTitle: Spark SQL Upgrading Guide
- In Spark version 2.4 and earlier
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r240006479
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -2563,4 +2563,18 @@ class JsonSuite extends
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23196
@srowen @HyukjinKwon @gatorsmile Could you take a look at the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r24694
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -37,6 +37,8 @@ displayTitle: Spark SQL Upgrading Guide
- In Spark version 2.4 and earlier
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23201#discussion_r24411
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonInferSchema.scala
---
@@ -121,7 +122,26 @@ private[sql] class JsonInferSchema
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23196#discussion_r24119
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -33,6 +33,8 @@ displayTitle: Spark SQL Upgrading Guide
- Spark applications which
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r239998397
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -37,6 +37,8 @@ displayTitle: Spark SQL Upgrading Guide
- In Spark version 2.4 and earlier
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23196#discussion_r239998126
--- Diff:
sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
---
@@ -49,8 +49,8 @@ class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23257
@gatorsmile Please, have a look at this PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23257
[SPARK-26310][SQL] Verify applicability of JSON options
## What changes were proposed in this pull request?
In the PR, I propose additional verification of JSON options. In
particular
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23253#discussion_r239853384
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -37,6 +37,8 @@ displayTitle: Spark SQL Upgrading Guide
- In Spark version 2.4 and earlier
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23253
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23253
[SPARK-26303][SQL] Return partial results for bad JSON records
## What changes were proposed in this pull request?
In the PR, I propose to return partial results from JSON datasource
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23201#discussion_r239547742
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonInferSchema.scala
---
@@ -121,7 +122,26 @@ private[sql] class JsonInferSchema
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23201#discussion_r239537694
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonInferSchema.scala
---
@@ -121,7 +122,26 @@ private[sql] class JsonInferSchema
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23201
@cloud-fan May I ask you to look at this PR, please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23202
@HyukjinKwon @srowen Is there anything which worries you in the PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23159#discussion_r239269414
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1777,7 +1777,7 @@ class Analyzer
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23201#discussion_r239269170
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JsonInferSchema.scala
---
@@ -121,7 +121,18 @@ private[sql] class JsonInferSchema
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23196
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23159
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23235#discussion_r239055208
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -35,6 +35,8 @@ displayTitle: Spark SQL Upgrading Guide
- Since Spark 3.0, CSV datasource
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23235
@cloud-fan Please, have a look at the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23235
[SPARK-26151][SQL][FOLLOWUP] Return partial results for bad CSV records
## What changes were proposed in this pull request?
Updated SQL migration guide according to changes in
https
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23120
The PR https://github.com/apache/spark/pull/23235 updates the sql migration
guide
---
-
To unsubscribe, e-mail: reviews
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23196#discussion_r239010321
--- Diff:
sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
---
@@ -49,8 +49,8 @@ class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23159
> Rather than change every single call to this method, if this should
generally be the value of the argument, then why not make it the default value
or something?
New parameter a
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23196#discussion_r238853314
--- Diff:
sql/hive/compatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
---
@@ -49,8 +49,8 @@ class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23159
@HyukjinKwon @dongjoon-hyun @srowen @zsxwing Do you have any objections of
this PR?
---
-
To unsubscribe, e-mail: reviews
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23202#discussion_r238696317
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVInferSchema.scala
---
@@ -98,6 +100,7 @@ class CSVInferSchema(options
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21363
@srowen There is another PR with related changes - inferring `DateType`
from CSV: https://github.com/apache/spark/pull/23202
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23202#discussion_r238194142
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVInferSchema.scala
---
@@ -98,6 +100,7 @@ class CSVInferSchema(options
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23196#discussion_r238116870
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -33,6 +33,8 @@ displayTitle: Spark SQL Upgrading Guide
- Spark applications which
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23202
[SPARK-26248][SQL] Infer date type from CSV
## What changes were proposed in this pull request?
The `CSVInferSchema` class is extended to support inferring of `DateType`
from CSV input
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23201
[SPARK-26246][SQL] Infer date and timestamp types from JSON
## What changes were proposed in this pull request?
The `JsonInferSchema` class is extended to support `DateType
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23196#discussion_r238111358
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -33,6 +33,8 @@ displayTitle: Spark SQL Upgrading Guide
- Spark applications which
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
@srowen I think this PR is ready.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23120#discussion_r238097513
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/UnivocityParser.scala
---
@@ -243,21 +243,27 @@ class UnivocityParser
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23150#discussion_r238075711
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/csv/UnivocityParserSuite.scala
---
@@ -86,62 +85,74 @@ class UnivocityParserSuite
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23150#discussion_r238075664
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -622,10 +623,11 @@ class CSVSuite extends QueryTest
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23150#discussion_r238075585
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1107,7 +,7 @@ class CSVSuite extends QueryTest
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
> they pass right? is there another test you were unable to add?
For now everything has been passed. I run all test localy on different
timezones (set via jvm parameter `-Duser.timez
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23196
[SPARK-26243][SQL] Use java.time API for parsing timestamps and dates from
JSON
## What changes were proposed in this pull request?
In the PR, I propose to switch on **java.time API
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23173#discussion_r238070641
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -171,15 +171,21 @@ private[csv] class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
> ... if it's ready from your side @MaxGekk
@srowen I just think how I could reduce number of changes in tests. In some
cases, test behavior depends on current time zone on my lap
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
@HyukjinKwon May I ask you to look at this PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23173#discussion_r238059569
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1987,6 +1987,21 @@ class CSVSuite extends
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23173#discussion_r238059480
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -171,15 +171,21 @@ private[csv] class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23159
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23120#discussion_r237873832
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/UnivocityParser.scala
---
@@ -243,21 +243,27 @@ class UnivocityParser
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23173#discussion_r237679872
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1987,6 +1987,18 @@ class CSVSuite extends
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23173#discussion_r237633755
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1987,6 +1987,18 @@ class CSVSuite extends
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23173#discussion_r237635061
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1987,6 +1987,18 @@ class CSVSuite extends
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23173#discussion_r237634432
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1987,6 +1987,18 @@ class CSVSuite extends
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23173#discussion_r237633638
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -1987,6 +1987,18 @@ class CSVSuite extends
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23132
@cloud-fan I did the same for CSV:
https://github.com/apache/spark/pull/22979
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23132
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22979
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23173
It seems this is similar to @HyukjinKwon PR:
https://github.com/apache/spark/pull/13252
---
-
To unsubscribe, e-mail: reviews
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23052#discussion_r237282738
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -169,13 +169,18 @@ private[csv] class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23159
@gatorsmile @cloud-fan Could you look at the changes - extracted from
another PR: https://github.com/apache/spark/pull/22429
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23132#discussion_r237239829
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -9,6 +9,8 @@ displayTitle: Spark SQL Upgrading Guide
## Upgrading From Spark SQL 2.4 to 3.0
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
Some tests didn't pass on new changes till I set time zone explicitly. The
tests use the same functions for checking correctness as the code that is
supposed to test. I think need more
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
I have to correct timestamp/date pattern in a few test to follow ISO 8601
(see [Patterns for Formatting and
Parsing](https://docs.oracle.com/javase/8/docs/api/java/time/format
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
> Are there any other behavior changes with the new code, besides being
able to parse microseconds?
The main one is new parser doesn't have the fallback to
`DateTimeUtils.stringToT
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
@srowen @HyukjinKwon @viirya @mgaido91 May I ask you to look at the PR. The
changes are related to another PR which you have reviewed already
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23052
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23120#discussion_r237093726
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/UnivocityParser.scala
---
@@ -243,21 +243,27 @@ class UnivocityParser
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23132#discussion_r237091555
--- Diff: docs/sql-migration-guide-upgrade.md ---
@@ -9,6 +9,8 @@ displayTitle: Spark SQL Upgrading Guide
## Upgrading From Spark SQL 2.4 to 3.0
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23132#discussion_r237091495
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -1610,6 +1610,13 @@ object SQLConf {
""&qu
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23130#discussion_r237085148
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/SaveLoadSuite.scala ---
@@ -142,4 +144,15 @@ class SaveLoadSuite extends DataSourceTest
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23120#discussion_r237065584
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -33,26 +33,21 @@ class FailureSafeParser
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23052
Actually it needs similar changes like in
https://github.com/apache/spark/pull/23130
---
-
To unsubscribe, e-mail: reviews
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23130#discussion_r237050065
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/SaveLoadSuite.scala ---
@@ -142,4 +144,15 @@ class SaveLoadSuite extends DataSourceTest
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23052
> seems like a real failure
I am looking at it. It seems the test is not deterministic.
---
-
To unsubscribe, e-m
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23120
@cloud-fan May I ask you to take a look at the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22429
Here is a PR introduces `maxFields` parameter to all function involved in
creation of truncated strings of spark plans:
https://github.com/apache/spark/pull/23159
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23159
ping @hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23159
[SPARK-26191][SQL] Control truncation of Spark plans via maxFields parameter
## What changes were proposed in this pull request?
In the PR, I propose to add `maxFields` parameter to all
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23132
@dongjoon-hyun @cloud-fan Please, take a look at the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23127
> jenkins retest this please
@juliuszsompolski It won't help, you need to fix python tests it seems
```
Failed example:
df.explain()
Differences (nd
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23130#discussion_r236719477
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/SaveLoadSuite.scala ---
@@ -142,4 +143,15 @@ class SaveLoadSuite extends DataSourceTest
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23052#discussion_r236659185
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -169,13 +169,18 @@ private[csv] class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23120
@HyukjinKwon Please, review the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23150
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22979
jenkins, retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23052#discussion_r236584201
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -169,13 +169,18 @@ private[csv] class
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21363
Here is the PR: https://github.com/apache/spark/pull/23150 which allows to
switch on java.time API (in CSV so far
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23150
[SPARK-26178][SQL] Use java.time API for parsing timestamps and dates from
CSV
## What changes were proposed in this pull request?
In the PR, I propose to use **java.time API
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/23132
[SPARK-26163][SQL] Parsing decimals from JSON using locale
## What changes were proposed in this pull request?
In the PR, I propose using of the locale option to parse (and infer
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23130
@cloud-fan @HyukjinKwon Please, take a look at the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22938#discussion_r236049216
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1892,7 +1898,7 @@ class JsonSuite extends
1 - 100 of 1029 matches
Mail list logo