Skip to content

Conversation

@candiduslynx
Copy link
Contributor

Follow-up for #823

@github-actions
Copy link

github-actions bot commented May 4, 2023

⏱️ Benchmark results

Comparing with 9426d11

  • DefaultConcurrencyDFS-2 resources/s: 11,907 ⬆️ 11.93% increase vs. 9426d11
  • DefaultConcurrencyRoundRobin-2 resources/s: 12,304 ⬆️ 0.17% increase vs. 9426d11
  • Glob-2 ns/op: 231 ⬆️ 31.34% increase vs. 9426d11
  • TablesWithChildrenDFS-2 resources/s: 27,569 ⬇️ 1.81% decrease vs. 9426d11
  • TablesWithChildrenRoundRobin-2 resources/s: 27,712 ⬆️ 7.33% increase vs. 9426d11
  • TablesWithRateLimitingDFS-2 resources/s: 28.4 ⬆️ 0.35% increase vs. 9426d11
  • TablesWithRateLimitingRoundRobin-2 resources/s: 825.2 ⬇️ 2.16% decrease vs. 9426d11
  • BufferedScanner-2 ns/op: 10.33 ⬆️ 10.78% increase vs. 9426d11
  • LogReader-2 ns/op: 32.68 ⬆️ 12.24% increase vs. 9426d11

@codecov
Copy link

codecov bot commented May 4, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.05 ⚠️

Comparison is base (9426d11) 47.13% compared to head (0510fa6) 47.09%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #847      +/-   ##
==========================================
- Coverage   47.13%   47.09%   -0.05%     
==========================================
  Files          76       76              
  Lines        7849     7857       +8     
==========================================
  Hits         3700     3700              
- Misses       3650     3654       +4     
- Partials      499      503       +4     
Impacted Files Coverage Δ
types/inet.go 41.08% <0.00%> (-0.65%) ⬇️
types/json.go 59.68% <0.00%> (-0.94%) ⬇️
types/mac.go 45.03% <0.00%> (-0.70%) ⬇️
types/uuid.go 43.07% <0.00%> (-0.68%) ⬇️

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

@candiduslynx candiduslynx changed the title fix(arrow): Allow nil valid param in AppendValues fix(arrow): Allow empty and nil valid param in AppendValues May 5, 2023
@github-actions github-actions bot added fix and removed fix labels May 5, 2023
@kodiakhq kodiakhq bot merged commit dafd05b into main May 5, 2023
@kodiakhq kodiakhq bot deleted the feat/arrow-extenstions-allow-nil-valid branch May 5, 2023 08:15
kodiakhq bot pushed a commit that referenced this pull request May 5, 2023
🤖 I have created a release *beep* *boop*
---


## [2.5.4](v2.5.3...v2.5.4) (2023-05-05)


### Bug Fixes

* **arrow:** Allow empty and `nil` valid param in `AppendValues` ([#847](#847)) ([dafd05b](dafd05b))

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