Skip to content

chore: Make proto public#842

Merged
kodiakhq[bot] merged 1 commit intomainfrom
chore/public_pb
May 2, 2023
Merged

chore: Make proto public#842
kodiakhq[bot] merged 1 commit intomainfrom
chore/public_pb

Conversation

@yevgenypats
Copy link
Contributor

This makes pb directory public as Im preparing for v3 and removing some code duplication that we had with versions around clients. Now old servers and clients will be able to just import an old version instead of copying all the old versions to the new SDK.

@github-actions
Copy link

github-actions bot commented May 1, 2023

⏱️ Benchmark results

Comparing with 47da73d

  • DefaultConcurrencyDFS-2 resources/s: 11,219 ⬇️ 5.95% decrease vs. 47da73d
  • DefaultConcurrencyRoundRobin-2 resources/s: 10,959 ⬇️ 9.22% decrease vs. 47da73d
  • Glob-2 ns/op: 169.9 ⬇️ 5.36% decrease vs. 47da73d
  • TablesWithChildrenDFS-2 resources/s: 28,177 ⬇️ 6.55% decrease vs. 47da73d
  • TablesWithChildrenRoundRobin-2 resources/s: 29,362 ⬆️ 4.54% increase vs. 47da73d
  • TablesWithRateLimitingDFS-2 resources/s: 28.17 ⬆️ 0.25% increase vs. 47da73d
  • TablesWithRateLimitingRoundRobin-2 resources/s: 823.4 ⬇️ 4.48% decrease vs. 47da73d
  • BufferedScanner-2 ns/op: 9.278 ⬇️ 0.08% decrease vs. 47da73d
  • LogReader-2 ns/op: 30.47 (no change)

@codecov
Copy link

codecov bot commented May 1, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (47da73d) 47.18% compared to head (cf249bd) 47.18%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #842   +/-   ##
=======================================
  Coverage   47.18%   47.18%           
=======================================
  Files          76       76           
  Lines        7841     7841           
=======================================
  Hits         3700     3700           
  Misses       3642     3642           
  Partials      499      499           
Impacted Files Coverage Δ
clients/destination/v0/destination.go 36.12% <ø> (ø)
clients/source/v0/source.go 29.68% <ø> (ø)
serve/destination.go 54.61% <ø> (ø)
serve/source.go 58.94% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@kodiakhq kodiakhq bot merged commit 6cbcd8e into main May 2, 2023
@kodiakhq kodiakhq bot deleted the chore/public_pb branch May 2, 2023 11:00
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