Skip to content

Clarify docs for df.to_sql chunksize #36172

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

Closed

Conversation

gbrova
Copy link

@gbrova gbrova commented Sep 6, 2020

Note: this is a docs pr, so not adding tests or a whatsnew entry.

@@ -2634,7 +2634,8 @@ def to_sql(
`index` is True, then the index names are used.
A sequence should be given if the DataFrame uses MultiIndex.
chunksize : int, optional
Specify the number of rows in each batch to be written at a time.
If batching is enabled via `method`, specify the number of rows in
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not overly familiar with our implementation but I think that this isn't tied to method as described here - are you seeing something definitely in the code that leads you to believe this?

@jorisvandenbossche

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I commented on the issue about it: #35891, let's discuss there further (but indeed it doesn't seem to depend on the method based on the code)

@WillAyd WillAyd added the Docs label Sep 9, 2020
@jorisvandenbossche jorisvandenbossche added the Needs Discussion Requires discussion from core team before further action label Sep 10, 2020
@WillAyd
Copy link
Member

WillAyd commented Sep 14, 2020

Thanks for the PR but looks like more discussion is needed in original issue before doing anything here

@WillAyd WillAyd closed this Sep 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Needs Discussion Requires discussion from core team before further action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC: df.to_sql chunksize seems to be ignored by default.
3 participants