Skip to content

fix: PK Addition Order#607

Merged
kodiakhq[bot] merged 3 commits intocloudquery:mainfrom
bbernays:change-PK-Addition-Order
Jan 14, 2023
Merged

fix: PK Addition Order#607
kodiakhq[bot] merged 3 commits intocloudquery:mainfrom
bbernays:change-PK-Addition-Order

Conversation

@bbernays
Copy link
Copy Markdown
Contributor

Summary

This will only add the default PKs once the Transformers have run and added any and all PKs


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 ✅

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 13, 2023

⏱️ Benchmark results

Comparing with f548a54

  • DefaultConcurrencyDFS-2 resources/s: 10,883 ⬇️ 6.77% decrease vs. f548a54
  • DefaultConcurrencyRoundRobin-2 resources/s: 12,147 ⬆️ 1.18% increase vs. f548a54
  • Glob-2 ns/op: 147.4 ⬆️ 0.27% increase vs. f548a54
  • TablesWithChildrenDFS-2 resources/s: 29,157 ⬇️ 8.52% decrease vs. f548a54
  • TablesWithChildrenRoundRobin-2 resources/s: 29,266 ⬆️ 6.08% increase vs. f548a54
  • TablesWithRateLimitingDFS-2 resources/s: 28.13 ⬇️ 0.53% decrease vs. f548a54
  • TablesWithRateLimitingRoundRobin-2 resources/s: 788.6 ⬇️ 4.45% decrease vs. f548a54
  • BufferedScanner-2 ns/op: 10.09 ⬆️ 0.50% increase vs. f548a54
  • LogReader-2 ns/op: 30.52 ⬇️ 0.03% decrease vs. f548a54

@disq
Copy link
Copy Markdown
Member

disq commented Jan 13, 2023

Add a test so we don't have a regression later?

@bbernays bbernays requested a review from disq January 13, 2023 20:26
@kodiakhq kodiakhq bot merged commit eff40e7 into cloudquery:main Jan 14, 2023
@bbernays bbernays deleted the change-PK-Addition-Order branch January 14, 2023 13:16
kodiakhq bot pushed a commit that referenced this pull request Jan 15, 2023
🤖 I have created a release *beep* *boop*
---


## [1.25.1](v1.25.0...v1.25.1) (2023-01-14)


### Bug Fixes

* Change options for new client ([#603](#603)) ([f548a54](f548a54))
* PK Addition Order ([#607](#607)) ([eff40e7](eff40e7))

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants