Skip to content

fix: Scalar set now accepts scalar type#902

Merged
kodiakhq[bot] merged 2 commits intomainfrom
fix/scalar
May 25, 2023
Merged

fix: Scalar set now accepts scalar type#902
kodiakhq[bot] merged 2 commits intomainfrom
fix/scalar

Conversation

@yevgenypats
Copy link
Copy Markdown
Contributor

This brings back functionally that we had in cqtypes which needs to keep all our resolver working (specifically the ParentColumnResolver)

@yevgenypats yevgenypats requested review from candiduslynx and disq May 25, 2023 10:06
@github-actions github-actions bot added the fix label May 25, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented May 25, 2023

Codecov Report

Patch coverage: 42.30% and project coverage change: +0.32 🎉

Comparison is base (f5f7cd7) 47.27% compared to head (d60b570) 47.60%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #902      +/-   ##
==========================================
+ Coverage   47.27%   47.60%   +0.32%     
==========================================
  Files          55       55              
  Lines        5028     5132     +104     
==========================================
+ Hits         2377     2443      +66     
- Misses       2401     2426      +25     
- Partials      250      263      +13     
Impacted Files Coverage Δ
scalar/binary.go 32.00% <0.00%> (-6.10%) ⬇️
scalar/scalar.go 4.12% <ø> (ø)
scalar/string.go 26.92% <0.00%> (-4.90%) ⬇️
scalar/bool.go 50.98% <50.00%> (+4.46%) ⬆️
scalar/float.go 43.95% <50.00%> (+2.99%) ⬆️
scalar/inet.go 54.16% <50.00%> (+1.89%) ⬆️
scalar/int.go 30.00% <50.00%> (+2.72%) ⬆️
scalar/json.go 55.75% <50.00%> (+1.46%) ⬆️
scalar/list.go 29.41% <50.00%> (+6.07%) ⬆️
scalar/mac.go 34.48% <50.00%> (+6.48%) ⬆️
... and 3 more

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

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 25, 2023

⏱️ Benchmark results

Comparing with f5f7cd7

  • DefaultConcurrencyDFS-2 resources/s: 10,851 ⬆️ 5.34% increase vs. f5f7cd7
  • DefaultConcurrencyRoundRobin-2 resources/s: 11,081 ⬇️ 9.50% decrease vs. f5f7cd7
  • Glob-2 ns/op: 211.3 ⬆️ 9.18% increase vs. f5f7cd7
  • TablesWithChildrenDFS-2 resources/s: 28,678 ⬇️ 0.14% decrease vs. f5f7cd7
  • TablesWithChildrenRoundRobin-2 resources/s: 28,328 ⬇️ 1.57% decrease vs. f5f7cd7
  • TablesWithRateLimitingDFS-2 resources/s: 28.24 ⬇️ 1.63% decrease vs. f5f7cd7
  • TablesWithRateLimitingRoundRobin-2 resources/s: 801.3 ⬇️ 7.49% decrease vs. f5f7cd7

@kodiakhq kodiakhq bot merged commit 1ff2229 into main May 25, 2023
@kodiakhq kodiakhq bot deleted the fix/scalar branch May 25, 2023 10:22
kodiakhq bot pushed a commit that referenced this pull request May 25, 2023
🤖 I have created a release *beep* *boop*
---


## [3.6.4](v3.6.3...v3.6.4) (2023-05-25)


### Bug Fixes

* Scalar set now accepts scalar type ([#902](#902)) ([1ff2229](1ff2229))

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