Skip to content

Commit 959d862

Browse files
author
Baz
authored
🐛 Source SalesForce: changed DEFAULT_WAIT_TIMEOUT_SECONDS to 24-hour limit (#15444)
1 parent 0092712 commit 959d862

File tree

6 files changed

+8
-7
lines changed

6 files changed

+8
-7
lines changed

airbyte-config/init/src/main/resources/seed/source_definitions.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -849,7 +849,7 @@
849849
- name: Salesforce
850850
sourceDefinitionId: b117307c-14b6-41aa-9422-947e34922962
851851
dockerRepository: airbyte/source-salesforce
852-
dockerImageTag: 1.0.11
852+
dockerImageTag: 1.0.12
853853
documentationUrl: https://docs.airbyte.io/integrations/sources/salesforce
854854
icon: salesforce.svg
855855
sourceType: api

airbyte-config/init/src/main/resources/seed/source_specs.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8272,7 +8272,7 @@
82728272
supportsNormalization: false
82738273
supportsDBT: false
82748274
supported_destination_sync_modes: []
8275-
- dockerImage: "airbyte/source-salesforce:1.0.11"
8275+
- dockerImage: "airbyte/source-salesforce:1.0.12"
82768276
spec:
82778277
documentationUrl: "https://docs.airbyte.com/integrations/sources/salesforce"
82788278
connectionSpecification:
@@ -8312,7 +8312,7 @@
83128312
title: "Refresh Token"
83138313
description: "Enter your application's <a href=\"https://developer.salesforce.com/docs/atlas.en-us.mobile_sdk.meta/mobile_sdk/oauth_refresh_token_flow.htm\"\
83148314
>Salesforce Refresh Token</a> used for Airbyte to access your Salesforce\
8315-
\ account. "
8315+
\ account."
83168316
type: "string"
83178317
airbyte_secret: true
83188318
order: 4

airbyte-integrations/connectors/source-salesforce/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,5 @@ RUN pip install .
1313

1414
ENTRYPOINT ["python", "/airbyte/integration_code/main.py"]
1515

16-
LABEL io.airbyte.version=1.0.11
16+
LABEL io.airbyte.version=1.0.12
1717
LABEL io.airbyte.name=airbyte/source-salesforce

airbyte-integrations/connectors/source-salesforce/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ docker run --rm -v $(pwd)/secrets:/secrets -v $(pwd)/integration_tests:/integrat
8181
Make sure to familiarize yourself with [pytest test discovery](https://docs.pytest.org/en/latest/goodpractices.html#test-discovery) to know how your test files and methods should be named.
8282
First install test dependencies into your virtual environment:
8383
```
84-
pip install .[tests]
84+
pip install ".[tests]"
8585
```
8686
### Unit Tests
8787
To run unit tests locally, from the connector directory run:

airbyte-integrations/connectors/source-salesforce/source_salesforce/streams.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,8 +149,8 @@ def read_records(
149149

150150

151151
class BulkSalesforceStream(SalesforceStream):
152-
page_size = 30000
153-
DEFAULT_WAIT_TIMEOUT_SECONDS = 600
152+
page_size = 15000
153+
DEFAULT_WAIT_TIMEOUT_SECONDS = 86400 # 24-hour bulk job running time
154154
MAX_CHECK_INTERVAL_SECONDS = 2.0
155155
MAX_RETRY_NUMBER = 3
156156

docs/integrations/sources/salesforce.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,7 @@ Now that you have set up the Salesforce source connector, check out the followin
119119

120120
| Version | Date | Pull Request | Subject |
121121
|:--------|:-----------|:-------------------------------------------------------------|:---------------------------------------------------------------------------------------------------------------------------------|
122+
| 1.0.12 | 2022-08-09 | [15444](https://github.com/airbytehq/airbyte/pull/15444) | Fixed bug when `Bulk Job` was timeout by the connector, but remained running on the server |
122123
| 1.0.11 | 2022-07-07 | [13729](https://github.com/airbytehq/airbyte/pull/13729) | Improve configuration field descriptions |
123124
| 1.0.10 | 2022-06-09 | [13658](https://github.com/airbytehq/airbyte/pull/13658) | Correct logic to sync stream larger than page size |
124125
| 1.0.9 | 2022-05-06 | [12685](https://github.com/airbytehq/airbyte/pull/12685) | Update CDK to v0.1.56 to emit an `AirbyeTraceMessage` on uncaught exceptions |

0 commit comments

Comments
 (0)