Skip to content
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

Unable to upload files >64 MB. #326

Closed
whatthestack opened this issue Aug 7, 2018 · 9 comments
Closed

Unable to upload files >64 MB. #326

whatthestack opened this issue Aug 7, 2018 · 9 comments

Comments

@whatthestack
Copy link

It would say the following error:
TypeError: publish_req_chunked() takes at most 3 arguments (4 given)
After jumping into the package, I found editing these lines resolved the issue:

server-client-python/tableauserverclient/server/request_factory.py - line 105 and 394, i added either connection_credentials or connections - depending on what was missing:

105 - Previous: def publish_req_chunked(self, datasource_item, connection_credentials=None):
105 - Fixed: def publish_req_chunked(self, datasource_item, connection_credentials=None, connections=None):

394 - Previous: def publish_req_chunked(self, workbook_item, connections=None):
394 - Fixed: def publish_req_chunked(self, workbook_item, connection_credentials=None, connections=None):

Is there a reason why it was left off? Looking at the following lines, it makes more sense to have them there.

After I made these changes, I was able to upload a 700+ MB file.

@t8y8
Copy link
Collaborator

t8y8 commented Aug 13, 2018

This is fixed in the development branch; can you pull from there and verify your issue is resolved?

bskim45 added a commit to bskim45/server-client-python that referenced this issue Aug 17, 2018
@bskim45
Copy link
Contributor

bskim45 commented Aug 17, 2018

I just made another PR to cover latter case.
btw, is there any ETA for next release? @t8y8

@pmehai
Copy link

pmehai commented Jan 16, 2019

Are there any updates on this issue? I'm running into this error when trying to upload an 84MB extract.

@bskim45
Copy link
Contributor

bskim45 commented Jan 17, 2019

The fix is merged but not released yet. You should install the package directly from development branch.

@t8y8
Copy link
Collaborator

t8y8 commented Jan 19, 2019

@RussTheAerialist I know an internal team has a PR out that they’d like released shortly, maybe we can try and get those merged and released next week. I’m on vacation but I’ll support code reviews that I’m tagged in!

@sharono011
Copy link

This is still broken. When will the fix be released?

@irwando
Copy link
Contributor

irwando commented Jul 25, 2019

Released. Please re-open if you still have issues.

@irwando irwando closed this as completed Jul 25, 2019
@samyak53
Copy link

samyak53 commented Jun 14, 2020

I am getting this error TypeError: publish_req_chunked() takes from 2 to 3 positional arguments but 4 were given when I try to publish datasource having size 300MB using this function server.datasources.publish(). @RussTheAerialist

@DuncanTormey
Copy link

I am getting this error TypeError: publish_req_chunked() takes from 2 to 3 positional arguments but 4 were given when I try to publish datasource having size 300MB using this function server.datasources.publish(). @RussTheAerialist

@samyak53 I am getting this error too. Were you able to resolve?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants