Skip to content

Commit

Permalink
mtuchi feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
daissatou2 committed Jul 24, 2023
1 parent 060df33 commit cff3839
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions adaptors/salesforce.md
Original file line number Diff line number Diff line change
Expand Up @@ -146,11 +146,6 @@ values (e.g., `M: male, F: female`).
semicolon-separated strings (e.g.,
`Services__c: 'Food;Counselling;Medical_Aid`).

#### Timeout considerations

The Salesforce adaptor currently waits for the batch to finish syncing to Salesforce to ensure that the batch results (i.e., whether the batch import was successful, how many records succeeded, how many failures) can be relayed back to the OpenFn user in Activity History. This can potentially cause timeout errors in OpenFn if the batch sync triggers Salesforce automation which takes long to complete.

In future adaptor updates, OpenFn will consider adding a new option to the adaptor to not wait for the batch results, and to have the batch run asynchronously. The tradeoff is that the OpenFn run would always succeed–and the administrator would have to log into Salesforce and monitor the batch results via the Salesforce Setup “Bulk Data Load Jobs” page. With this approach, OpenFn wouldn’t set limits on the batch processing time avoiding timeout errors, but it does move the batch monitoring from OpenFn to Salesforce.

### Salesforce Credentials

Expand Down

0 comments on commit cff3839

Please sign in to comment.