Skip to content

CLN: read_sql date parsing #24500

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Dec 31, 2018
Merged

Conversation

mroeschke
Copy link
Member

@mroeschke mroeschke commented Dec 30, 2018

Cleaning up the date parsing portion of the read_sql functionality. Also checking that #15119 can be closed as we are already testing the solution.

@codecov
Copy link

codecov bot commented Dec 30, 2018

Codecov Report

Merging #24500 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #24500   +/-   ##
=======================================
  Coverage    92.3%    92.3%           
=======================================
  Files         166      166           
  Lines       52412    52412           
=======================================
  Hits        48381    48381           
  Misses       4031     4031
Flag Coverage Δ
#multiple 90.73% <ø> (ø) ⬆️
#single 43.05% <ø> (-0.01%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 681e75c...d84c5c0. Read the comment docs.

@codecov
Copy link

codecov bot commented Dec 30, 2018

Codecov Report

Merging #24500 into master will decrease coverage by 60.41%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master   #24500       +/-   ##
===========================================
- Coverage    92.3%   31.88%   -60.42%     
===========================================
  Files         166      166               
  Lines       52412    52421        +9     
===========================================
- Hits        48381    16717    -31664     
- Misses       4031    35704    +31673
Flag Coverage Δ
#multiple 30.29% <ø> (-60.44%) ⬇️
#single 31.88% <ø> (-11.17%) ⬇️
Impacted Files Coverage Δ
pandas/io/formats/latex.py 0% <0%> (-100%) ⬇️
pandas/core/categorical.py 0% <0%> (-100%) ⬇️
pandas/io/sas/sas_constants.py 0% <0%> (-100%) ⬇️
pandas/tseries/plotting.py 0% <0%> (-100%) ⬇️
pandas/tseries/converter.py 0% <0%> (-100%) ⬇️
pandas/io/formats/html.py 0% <0%> (-98.65%) ⬇️
pandas/core/groupby/categorical.py 0% <0%> (-95.46%) ⬇️
pandas/core/reshape/reshape.py 8.06% <0%> (-91.51%) ⬇️
pandas/io/sas/sas7bdat.py 0% <0%> (-91.17%) ⬇️
pandas/io/sas/sas_xport.py 0% <0%> (-90.15%) ⬇️
... and 126 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 681e75c...4fee9db. Read the comment docs.

@jreback jreback added Datetime Datetime data dtype IO SQL to_sql, read_sql, read_sql_query labels Dec 30, 2018
@jreback
Copy link
Contributor

jreback commented Dec 30, 2018

looks fine. is there any user api change here? (meaning do we need a whatsnew note)

@mroeschke
Copy link
Member Author

No API change at all, just reorganization. The linked issue already had an existing test, but the test wasn't as strict as it should have been.

@mroeschke mroeschke added this to the 0.24.0 milestone Dec 31, 2018
@jreback jreback merged commit 449d665 into pandas-dev:master Dec 31, 2018
@jreback
Copy link
Contributor

jreback commented Dec 31, 2018

thanks!

@mroeschke mroeschke deleted the datetime_sql_clean branch December 31, 2018 17:35
Pingviinituutti pushed a commit to Pingviinituutti/pandas that referenced this pull request Feb 28, 2019
Pingviinituutti pushed a commit to Pingviinituutti/pandas that referenced this pull request Feb 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Datetime Datetime data dtype IO SQL to_sql, read_sql, read_sql_query
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsistent parsing for timestamp with timezone with read_sql_query
2 participants