Skip to content

feat: Add scheduler option to AWS#13757

Merged
kodiakhq[bot] merged 3 commits intomainfrom
add-scheduler-to-aws
Sep 7, 2023
Merged

feat: Add scheduler option to AWS#13757
kodiakhq[bot] merged 3 commits intomainfrom
add-scheduler-to-aws

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

This wasn't moved into the AWS plugin before because there wasn't a strong need for it, but apparently some APIs (like account contacts) can benefit from this scheduler.

@hermanschaaf hermanschaaf requested review from bbernays and removed request for yevgenypats September 7, 2023 15:51
@cq-bot cq-bot added aws labels Sep 7, 2023
@bbernays bbernays added the automerge Automatically merge once required checks pass label Sep 7, 2023
TableOptions *tableoptions.TableOptions `json:"table_options,omitempty"`
Concurrency int `json:"concurrency"`
EventBasedSync *EventBasedSync `json:"event_based_sync,omitempty"`
Scheduler scheduler.Strategy `json:"scheduler,omitempty"`
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The default is DFS (it's an enum with zero value) so we don't need to set a default explicitly

@kodiakhq kodiakhq bot merged commit 521918f into main Sep 7, 2023
@kodiakhq kodiakhq bot deleted the add-scheduler-to-aws branch September 7, 2023 16:01
kodiakhq bot pushed a commit that referenced this pull request Sep 7, 2023
🤖 I have created a release *beep* *boop*
---


## [22.10.0](plugins-source-aws-v22.9.0...plugins-source-aws-v22.10.0) (2023-09-07)


### Features

* Add scheduler option to AWS ([#13757](#13757)) ([521918f](521918f))


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.7.1 ([#13713](#13713)) ([73004dc](73004dc))
* Issue [#13433](#13433) - match on lowercase "redirect" in aws policy  ([#13751](#13751)) ([539e029](539e029))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants