Skip to content

Conversation

@yevgenypats
Copy link
Contributor

Reverts #878

As far as I can see there is no need for this because over the wire it is always flattened anyway and in general I think we should move to always flatten structure as it causes more harm then good.

@yevgenypats yevgenypats changed the title Revert "feat(test): Test writing to a child table" feat: Revert "feat(test): Test writing to a child table" May 16, 2023
@github-actions github-actions bot added the feat label May 16, 2023
@github-actions
Copy link

⏱️ Benchmark results

Comparing with cb639cd

  • DefaultConcurrencyDFS-2 resources/s: 10,218 ⬇️ 1.76% decrease vs. cb639cd
  • DefaultConcurrencyRoundRobin-2 resources/s: 12,433 ⬆️ 9.70% increase vs. cb639cd
  • Glob-2 ns/op: 277.7 ⬆️ 27.66% increase vs. cb639cd
  • TablesWithChildrenDFS-2 resources/s: 24,255 ⬇️ 8.91% decrease vs. cb639cd
  • TablesWithChildrenRoundRobin-2 resources/s: 23,320 ⬇️ 18.34% decrease vs. cb639cd
  • TablesWithRateLimitingDFS-2 resources/s: 28.14 (no change)
  • TablesWithRateLimitingRoundRobin-2 resources/s: 813.8 ⬇️ 3.51% decrease vs. cb639cd

@yevgenypats yevgenypats merged commit 9d61013 into main May 16, 2023
@yevgenypats yevgenypats deleted the revert-878-feat/migrate-relation-test branch May 16, 2023 13:29
kodiakhq bot pushed a commit that referenced this pull request May 16, 2023
🤖 I have created a release *beep* *boop*
---


## [3.5.0](v3.4.0...v3.5.0) (2023-05-16)


### Features

* Revert "feat(test): Test writing to a child table" ([#880](#880)) ([9d61013](9d61013))

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

2 participants