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 srowen commented on the issue:
https://github.com/apache/spark/pull/21363
@MaxGekk now that your change is merge, can this proceed, @xuanyuanking ?
or is it obsolete?
---
-
To unsubscribe, e-mail:
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).
---
-
To
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/21363
@MaxGekk Sorry for the late, something inserted in the my scheduler, I plan
to start this PR in this weekend, if its too late please just take it, sorry
for the late again.
---
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21363
@xuanyuanking Are you still working on this? If not, I could continue.
---
-
To unsubscribe, e-mail:
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/21363
@HyukjinKwon Great thanks for ping me, I'll try to work on this and cc all
reviewer in this PR.
---
-
To unsubscribe,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21363
I am going to suggest to close this since it's being active more then few
weeks. It should be good to fix. Let me leave some cc's who might be interested
in this just FYI. Feel free to take
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21363
Looks difficult because the behaviours themselves are different. One
possibility is a fallback and the other possibility is configuration.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21363
**[Test build #91599 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91599/testReport)**
for PR 21363 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91599/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21363
**[Test build #91599 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91599/testReport)**
for PR 21363 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21363
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21363
@sergey-rubtsov we have to keep backward compatibility. If a user upgrades,
with your change a running application may break because of data not being
anymore timestamp but date. We can add a new
Github user sergey-rubtsov commented on the issue:
https://github.com/apache/spark/pull/21363
@HyukjinKwon please, clarify, how to add a configuration to control this
behaviour? Do you mean to keep backward compatibility?
But it seems, now, we don't handle a Spark "DataType" at
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21363
@sergey-rubtsov, would we be able to add a configuration to control this
behaviour? Sounds we should better have a configuration to control this
behaviour for now since the date / timestamp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
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/21363
**[Test build #90870 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90870/testReport)**
for PR 21363 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90870/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21363
**[Test build #90870 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90870/testReport)**
for PR 21363 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21363
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sergey-rubtsov commented on the issue:
https://github.com/apache/spark/pull/21363
cc @viirya @mgaido91 @dongjoon-hyun
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user sergey-rubtsov commented on the issue:
https://github.com/apache/spark/pull/21363
@HyukjinKwon, please take a look
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21363
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user sergey-rubtsov commented on the issue:
https://github.com/apache/spark/pull/21363
Previous pull request https://github.com/apache/spark/pull/20140 is closed
---
-
To unsubscribe, e-mail:
33 matches
Mail list logo