Skip to content

feat(aws): Use PreResourceResolver when necessary, remove ListAndDetailResolver#2460

Merged
kodiakhq[bot] merged 17 commits intocloudquery:mainfrom
disq:feat/aws_use_preresourceresolvers
Oct 12, 2022
Merged

feat(aws): Use PreResourceResolver when necessary, remove ListAndDetailResolver#2460
kodiakhq[bot] merged 17 commits intocloudquery:mainfrom
disq:feat/aws_use_preresourceresolvers

Conversation

@disq
Copy link
Copy Markdown
Member

@disq disq commented Oct 5, 2022

More here #2461

BEWARE merging this before updating the plugin-sdk for "PreResourceResolver parallelism" will cause performance regressions. (#2461 doesn't have that issue, as those resources are already linear)

@cq-bot cq-bot added the aws label Oct 5, 2022
@disq disq requested review from amanenk and bbernays October 5, 2022 17:35
@disq disq force-pushed the feat/aws_use_preresourceresolvers branch 4 times, most recently from 0df8cb5 to 10b1b05 Compare October 7, 2022 07:26
@yevgenypats
Copy link
Copy Markdown
Contributor

This should go in with cloudquery/plugin-sdk#268

@disq disq force-pushed the feat/aws_use_preresourceresolvers branch 2 times, most recently from fa2f3a0 to b8e34f6 Compare October 12, 2022 08:54
@disq
Copy link
Copy Markdown
Member Author

disq commented Oct 12, 2022

Now waiting for #2661

@disq disq force-pushed the feat/aws_use_preresourceresolvers branch from b8e34f6 to af1fc5a Compare October 12, 2022 09:26
Copy link
Copy Markdown
Contributor

@hermanschaaf hermanschaaf left a comment

Choose a reason for hiding this comment

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

👍

@disq disq added automerge Automatically merge once required checks pass priority merge labels Oct 12, 2022
@kodiakhq kodiakhq bot merged commit 340f614 into cloudquery:main Oct 12, 2022
@disq disq deleted the feat/aws_use_preresourceresolvers branch October 12, 2022 11:33
kodiakhq bot pushed a commit that referenced this pull request Oct 12, 2022
🤖 I have created a release *beep* *boop*
---


## [2.3.0](plugins-source-aws-v2.2.0...plugins-source-aws-v2.3.0) (2022-10-12)


### Features

* **aws:** Use PreResourceResolver when necessary, remove ListAndDetailResolver ([#2460](#2460)) ([340f614](340f614))

---
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.

4 participants