feature(connectors/rillet): add new Rillet connector example#556
feature(connectors/rillet): add new Rillet connector example#556jacksonbailey-maker wants to merge 23 commits intofivetran:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new Connector SDK example for syncing Rillet accounting resources, including connector implementation, configuration template, and documentation.
Changes:
- Added
connector.pyimplementing schema generation, incremental/paginated sync, retries/backoff, and checkpointing. - Added
configuration.jsontemplate for connector configuration. - Added example documentation (
README.md) and a local validation harness (test_connector.py). - Added a root
CLAUDE.mdwith AI assistant instructions.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 26 comments.
Show a summary per file
| File | Description |
|---|---|
connectors/rillet/connector.py |
New Rillet connector example with pagination/incremental logic, retry handling, and state checkpointing. |
connectors/rillet/configuration.json |
New configuration template for API key/base URL/version. |
connectors/rillet/README.md |
New user-facing documentation for running/configuring the example. |
connectors/rillet/test_connector.py |
New local “mock SDK” harness to validate basic import/execution. |
CLAUDE.md |
New top-level AI assistant instruction file. |
…sponse status codes.
jacksonbailey-maker
left a comment
There was a problem hiding this comment.
I've made all of the requested changes
fivetran-sahilkhirwal
left a comment
There was a problem hiding this comment.
Hi @jacksonbailey-maker
Can you please address the existing minor comments :)
Apart from them, the example looks good
Co-authored-by: Sahil Khirwal <sahil.khirwal@fivetran.com>
Co-authored-by: Sahil Khirwal <sahil.khirwal@fivetran.com>
Co-authored-by: Sahil Khirwal <sahil.khirwal@fivetran.com>
Co-authored-by: Sahil Khirwal <sahil.khirwal@fivetran.com>
fivetran-JenasVimal
left a comment
There was a problem hiding this comment.
LGTM
Just add testing details by uploading screenshots of fivetran debug and fivetran deploy
fivetran-chinmayichandrasekar
left a comment
There was a problem hiding this comment.
Left a few suggestions. I also think Co-pilot's review comments are valid, for example, including the Authentication section.
Co-authored-by: fivetran-chinmayichandrasekar <chinmayi.chandrasekar@fivetran.com>
fivetran-chinmayichandrasekar
left a comment
There was a problem hiding this comment.
LGTM


Jira ticket
Unable to create ticket due to existing account not being granted access to Jira on fivetran.atlassian.net
Description of Change
Summary
This PR adds a new Rillet connector example to the repo, including full connector implementation, configuration template, README documentation, and a local connector test harness. The user will need to provide their own API key in the
configuration.jsonfile, alone with specifying theapi_versionif using one other than version 3.Files added
What changed
validate_configuration()for required config checksschema()derived from endpoint metadataupdate()sync loop over Rillet collection endpoints_make_request()with retries, exponential backoff, and 429/5xx handlingupdated.gt, and checkpointingvendor-credits404sapi_keyREADME.md:Added
test_connector.pyto validate connector structure and local debug logic without the real SDK.Testing
connector.pyvia Python compile check.test_connector.pyis included to allow local validation of connector import and logic without requiring Fivetran runtime.Checklist
Some tips and links to help validate your PR:
fivetran debugcommand.