Skip to content

Conversation

@erezrokah
Copy link
Member

Summary

Related to cloudquery/cloudquery#7165. This will require a minor code change to the CLI to call GetDestinationByName here and here


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

@erezrokah erezrokah force-pushed the feat/spec_dest_sources_order branch from e26f419 to 8b289cd Compare January 25, 2023 09:30
@github-actions
Copy link

github-actions bot commented Jan 25, 2023

⏱️ Benchmark results

  • DefaultConcurrencyDFS-2 resources/s: 10,794
  • DefaultConcurrencyRoundRobin-2 resources/s: 11,880
  • Glob-2 ns/op: 183
  • TablesWithChildrenDFS-2 resources/s: 27,631
  • TablesWithChildrenRoundRobin-2 resources/s: 29,259
  • TablesWithRateLimitingDFS-2 resources/s: 28.22
  • TablesWithRateLimitingRoundRobin-2 resources/s: 827.6
  • BufferedScanner-2 ns/op: 9.516
  • LogReader-2 ns/op: 30.48

@erezrokah erezrokah force-pushed the feat/spec_dest_sources_order branch from 8b289cd to 59f0564 Compare January 31, 2023 11:09
@erezrokah erezrokah force-pushed the feat/spec_dest_sources_order branch from f1c437f to caf515b Compare February 23, 2023 08:59
@kodiakhq kodiakhq bot merged commit 4602071 into cloudquery:main Feb 23, 2023
@erezrokah erezrokah deleted the feat/spec_dest_sources_order branch February 23, 2023 09:05
kodiakhq bot pushed a commit that referenced this pull request Feb 23, 2023
🤖 I have created a release *beep* *boop*
---


## [1.40.0](v1.39.1...v1.40.0) (2023-02-23)


### Features

* **spec:** Return sources, destinations in order ([#624](#624)) ([4602071](4602071))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
kodiakhq bot pushed a commit to cloudquery/cloudquery that referenced this pull request Feb 28, 2023
…8403)



#### Summary

Fixes #7165 (the fix is in the SDK cloudquery/plugin-sdk#624).

The changes in the CLI are due to the source reader struct changes

<!--
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants