ckanapi icon indicating copy to clipboard operation
ckanapi copied to clipboard

Streaming uploads revisited

Open jqnatividad opened this issue 5 years ago • 5 comments

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 :) ).

jqnatividad avatar Jan 05 '21 13:01 jqnatividad

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

wardi avatar Apr 27 '21 20:04 wardi

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

frafra avatar Feb 01 '22 14:02 frafra

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

wardi avatar Feb 01 '22 16:02 wardi

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

ChristianF88 avatar Mar 02 '23 16:03 ChristianF88

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

wardi avatar Mar 02 '23 16:03 wardi