ckan / ckanapi

A command line interface and Python module for accessing the CKAN Action API
Other
181 stars 74 forks source link

Streaming uploads revisited #180

Open jqnatividad opened 3 years ago

jqnatividad commented 3 years ago

Hi @wardi. Happy New Year! I adapted @hvwaldow's old streaming uploads PR (#109), and can confirm it works like a charm!

Exercised it heavily with multi-gigabyte uploads both from the CLI and as a python module, and it performs as expected.

Submitting the PR again as I think it will be a valuable addition to ckanapi (and I'd rather not maintain my own fork :) ).

wardi commented 3 years ago

is the progress bar compatible with LocalCKAN (or ckanapi -c)? If not the switch should be moved together with the --insecure flag in the CLI which is only available when using ckanapi -r

frafra commented 2 years ago

Can we move [-P] and accept the change as it is?

wardi commented 2 years ago

Would be nice to have a test for this functionality as well.

ChristianF88 commented 1 year ago

Hi guys! I'm a quite new ckan user / dev. And I was wondering if there's any plan to merge this soon? Cheers!

wardi commented 1 year ago

@ChristianF88 there are some small things to fix I pointed out above, and some new conflicts.