Skip to content

fix: TablesForSpec should only return top-level tables#456

Merged
kodiakhq[bot] merged 2 commits intomainfrom
get-tables-for-spec-no-flat-tables
Dec 5, 2022
Merged

fix: TablesForSpec should only return top-level tables#456
kodiakhq[bot] merged 2 commits intomainfrom
get-tables-for-spec-no-flat-tables

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 2, 2022

⏱️ Benchmark results

  • DefaultConcurrency-2 resources/s: 11,343
  • Glob-2 ns/op: 147.5
  • TablesWithChildrenDefaultConcurrency-2 resources/s: 30,282
  • BufferedScanner-2 ns/op: 9.291
  • LogReader-2 ns/op: 30.54

Copy link
Copy Markdown
Contributor

@yevgenypats yevgenypats left a comment

Choose a reason for hiding this comment

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

Looks good. It includes the relations in the parent table when going over the wire right?

@hermanschaaf
Copy link
Copy Markdown
Contributor Author

It includes the relations in the parent table when going over the wire right?

@yevgenypats Right 👍

@kodiakhq kodiakhq bot merged commit ab7ca97 into main Dec 5, 2022
@kodiakhq kodiakhq bot deleted the get-tables-for-spec-no-flat-tables branch December 5, 2022 09:56
kodiakhq bot pushed a commit that referenced this pull request Dec 6, 2022
🤖 I have created a release *beep* *boop*
---


## [1.11.0](v1.10.0...v1.11.0) (2022-12-05)


### Features

* Add Support for net.IP in Faker ([#445](#445)) ([2deced1](2deced1))


### Bug Fixes

* **deps:** Update golang.org/x/exp digest to 6ab00d0 ([#449](#449)) ([b981e33](b981e33))
* **deps:** Update module github.com/avast/retry-go/v4 to v4.3.1 ([#450](#450)) ([e4116f1](e4116f1))
* **deps:** Update module google.golang.org/grpc to v1.51.0 ([#451](#451)) ([538211c](538211c))
* Don't log start-and-finish of relational tables ([#459](#459)) ([4d6eeca](4d6eeca))
* Sync Metrics for Long running plugins ([#455](#455)) ([3fecc61](3fecc61))
* TablesForSpec should only return top-level tables ([#456](#456)) ([ab7ca97](ab7ca97))

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

4 participants