[GitHub] [spark] HyukjinKwon commented on pull request #32558: [SPARK-34953][CORE][SQL] Add the code change for adding the DateType in the infer schema while reading in CSV and JSON

2022-08-10 Thread GitBox


HyukjinKwon commented on PR #32558:
URL: https://github.com/apache/spark/pull/32558#issuecomment-1211520074

   It's been added at https://github.com/apache/spark/pull/37327 and 
https://github.com/apache/spark/pull/23202


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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



[GitHub] [spark] HyukjinKwon commented on pull request #32558: [SPARK-34953][CORE][SQL] Add the code change for adding the DateType in the infer schema while reading in CSV and JSON

2021-07-21 Thread GitBox


HyukjinKwon commented on pull request #32558:
URL: https://github.com/apache/spark/pull/32558#issuecomment-884646343


   I would defer to @MaxGekk


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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



[GitHub] [spark] HyukjinKwon commented on pull request #32558: [SPARK-34953][CORE][SQL] Add the code change for adding the DateType in the infer schema while reading in CSV and JSON

2021-06-01 Thread GitBox


HyukjinKwon commented on pull request #32558:
URL: https://github.com/apache/spark/pull/32558#issuecomment-852628474


   Looks making sense to me.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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



[GitHub] [spark] HyukjinKwon commented on pull request #32558: [SPARK-34953][CORE][SQL] Add the code change for adding the DateType in the infer schema while reading in CSV and JSON

2021-05-31 Thread GitBox


HyukjinKwon commented on pull request #32558:
URL: https://github.com/apache/spark/pull/32558#issuecomment-851750660


   Oh I meant this: 
https://github.com/apache/spark/blob/master/python/pyspark/sql/readwriter.py#L342-L350
   These options are listed up as a parameter in Python side specifically. For 
CSV documentation, it's merged at https://github.com/apache/spark/pull/32658 so 
you could add the option in that page.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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



[GitHub] [spark] HyukjinKwon commented on pull request #32558: [SPARK-34953][CORE][SQL] Add the code change for adding the DateType in the infer schema while reading in CSV and JSON

2021-05-31 Thread GitBox


HyukjinKwon commented on pull request #32558:
URL: https://github.com/apache/spark/pull/32558#issuecomment-851278594


   I think we should also add the parameter at `csv` and `json` at 
`DataFrameWriter`, `DataFrameReader`, `DataStreamReader`, `DataStreamWriter` 
(and also `readwriter.py` and `streaming.py`). @itholic can you help review 
this one please?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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