-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[WIP] Salesforce does not download all results for large result set #1685
Comments
@dacort Sorry about this. I wrote the bulk of Salesforce.py. Most of my salesforce queries involve the salesforce version of joins and so I was never able to use/test batch results well. Thanks for your desire to improve this module! |
No problem @dlstadther! Happy to be able to contribute. :) |
Just realized this creates a CSV file with multiple headers. This shouldn't be merged in yet. |
Once things are fixed, i'll review once more |
* #1685 Add support for multiple results in batch jobs Adds a new method 'get_batch_result_ids' and maintains backwards compatibility with the old method, adding a warning. * Add the ability to merge result sets and basic testing. Note that this only supports CSV since this is what the library does by default.
For particularly large result sets, Salesforce will return multiple result IDs for a single batch.
However, in
get_batch_results
, only the first result ID is being returned. The code should utilizeET.findall
to return a list of result IDs and download each of those to append to the output file.I'm planning on submitting a PR for this in the next couple of days.
The text was updated successfully, but these errors were encountered: