Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adds the ability to Append a zero valued time.Time #1228

Merged
merged 2 commits into from
Mar 11, 2024
Merged

Adds the ability to Append a zero valued time.Time #1228

merged 2 commits into from
Mar 11, 2024

Conversation

KevinJoiner
Copy link
Contributor

@KevinJoiner KevinJoiner commented Mar 7, 2024 •
edited
Loading

Summary

Ignores date overflow check if the time.Time is a zero value.
#1227

Checklist

Delete items not relevant to your PR:

  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG
  • For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@CLAassistant
Copy link

CLAassistant commented Mar 7, 2024 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@jkaflik
Copy link
Contributor

jkaflik commented Mar 8, 2024

Hi @KevinJoiner
Thanks for your contribution. Could you add a test case?

@jkaflik jkaflik self-requested a review March 8, 2024 06:04
@KevinJoiner
Copy link
Contributor Author

@jkaflik I updated the change only to affect DateTime since the ch-go version does not include this PR yet ClickHouse/ch-go#392

Also, the integration tests added will need to be updated when the ch-go version includes this PR ClickHouse/ch-go#390

Not sure if it is optimal to merge this now or wait for the ch-go version to be updated.

@jkaflik
Copy link
Contributor

jkaflik commented Mar 11, 2024

@KevinJoiner ch-go changes are already merged and released. Let's update this PR accordingly.

@KevinJoiner
Copy link
Contributor Author

@jkaflik I updated Unit tests and Integration tests and squashed everything into 2 commits.

@jkaflik jkaflik self-assigned this Mar 11, 2024
@jkaflik jkaflik merged commit 7e2272c into ClickHouse:main Mar 11, 2024
12 checks passed
mx-psi referenced this pull request in open-telemetry/opentelemetry-collector-contrib Mar 12, 2024
@renovate @opentelemetrybot
fix(deps): update module github.com/clickhouse/clickhouse-go/v2 to v2…
3b205c8 
….21.1 (#31065)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/ClickHouse/clickhouse-go/v2](https://togithub.com/ClickHouse/clickhouse-go)
| `v2.17.1` -> `v2.21.1` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.17.1/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.17.1/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>ClickHouse/clickhouse-go
(github.com/ClickHouse/clickhouse-go/v2)</summary>

###
[`v2.21.1`](https://togithub.com/ClickHouse/clickhouse-go/releases/tag/v2.21.1)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.21.0...v2.21.1)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

##### Fixes 🐛

- Adds the ability to Append a zero valued time.Time by
[@&#8203;KevinJoiner](https://togithub.com/KevinJoiner) in
[https://github.com/ClickHouse/clickhouse-go/pull/1228](https://togithub.com/ClickHouse/clickhouse-go/pull/1228)

#### New Contributors

- [@&#8203;KevinJoiner](https://togithub.com/KevinJoiner) made their
first contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1228](https://togithub.com/ClickHouse/clickhouse-go/pull/1228)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.21.0...v2.21.1

###
[`v2.21.0`](https://togithub.com/ClickHouse/clickhouse-go/releases/tag/v2.21.0)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.20.0...v2.21.0)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

##### Enhancements 🎉

- Stream HTTP response body read for decompression by
[@&#8203;rogeryk](https://togithub.com/rogeryk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1213](https://togithub.com/ClickHouse/clickhouse-go/pull/1213)

##### Fixes 🐛

- Fix Date32 extremes values handling by
[@&#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1218](https://togithub.com/ClickHouse/clickhouse-go/pull/1218)
- Fix column with double quotes PrepareBatch
failed.([#&#8203;1216](https://togithub.com/ClickHouse/clickhouse-go/issues/1216))
by [@&#8203;YenchangChan](https://togithub.com/YenchangChan) in
[https://github.com/ClickHouse/clickhouse-go/pull/1217](https://togithub.com/ClickHouse/clickhouse-go/pull/1217)
- Fix an obscure error in HTTP protocol with LZ4 compression by
[@&#8203;YenchangChan](https://togithub.com/YenchangChan) in
[https://github.com/ClickHouse/clickhouse-go/pull/1230](https://togithub.com/ClickHouse/clickhouse-go/pull/1230)

##### Other Changes 🛠

- Replace deprecated `io/ioutil` functions with equivalents by
[@&#8203;zaneli](https://togithub.com/zaneli) in
[https://github.com/ClickHouse/clickhouse-go/pull/1211](https://togithub.com/ClickHouse/clickhouse-go/pull/1211)
- Run cloud workflow on main branch only by
[@&#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1219](https://togithub.com/ClickHouse/clickhouse-go/pull/1219)
- ClickHouse 24.2 by [@&#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1220](https://togithub.com/ClickHouse/clickhouse-go/pull/1220)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.20.0...v2.21.0

###
[`v2.20.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2200-2024-02-28----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.19.0...v2.20.0)

#### What's Changed

##### Enhancements 🎉

- Support \[n]byte/\[]byte type Scan/Append to FixedString column by
[@&#8203;rogeryk](https://togithub.com/rogeryk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1205](https://togithub.com/ClickHouse/clickhouse-go/pull/1205)

##### Other Changes 🛠

- Enable cloud tests by [@&#8203;jkaflik](https://togithub.com/jkaflik)
in
[https://github.com/ClickHouse/clickhouse-go/pull/1202](https://togithub.com/ClickHouse/clickhouse-go/pull/1202)
- Removed LowCardinality(UInt64) tests that caused
allow_suspicious_low_cardinality_types related error by
[@&#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1206](https://togithub.com/ClickHouse/clickhouse-go/pull/1206)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.19.0...v2.20.0

###
[`v2.19.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2190-2024-02-26----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.18.0...v2.19.0)

#### What's Changed

##### Enhancements 🎉

- handle ctx.Done() in acquire by
[@&#8203;threadedstream](https://togithub.com/threadedstream) in
[https://github.com/ClickHouse/clickhouse-go/pull/1199](https://togithub.com/ClickHouse/clickhouse-go/pull/1199)

##### Fixes 🐛

- Fix panic on format nil \*fmt.Stringer type value by
[@&#8203;zaneli](https://togithub.com/zaneli) in
[https://github.com/ClickHouse/clickhouse-go/pull/1200](https://togithub.com/ClickHouse/clickhouse-go/pull/1200)

##### Other Changes 🛠

- Update Go/ClickHouse versions by
[@&#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1201](https://togithub.com/ClickHouse/clickhouse-go/pull/1201)

#### New Contributors

- [@&#8203;threadedstream](https://togithub.com/threadedstream) made
their first contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1199](https://togithub.com/ClickHouse/clickhouse-go/pull/1199)
- [@&#8203;zaneli](https://togithub.com/zaneli) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1200](https://togithub.com/ClickHouse/clickhouse-go/pull/1200)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.18.0...v2.19.0

###
[`v2.18.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2180-2024-02-01----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.17.1...v2.18.0)

#### What's Changed

##### Enhancements 🎉

- Add WithAllocBufferColStrProvider string column allocator for batch
insert performance boost by
[@&#8203;hongker](https://togithub.com/hongker) in
[https://github.com/ClickHouse/clickhouse-go/pull/1181](https://togithub.com/ClickHouse/clickhouse-go/pull/1181)

##### Fixes 🐛

- Fix bind for seconds scale DateTime by
[@&#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1184](https://togithub.com/ClickHouse/clickhouse-go/pull/1184)

##### Other Changes 🛠

- resolves
[#&#8203;1163](https://togithub.com/ClickHouse/clickhouse-go/issues/1163)
debugF function is not respected by
[@&#8203;omurbekjk](https://togithub.com/omurbekjk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1166](https://togithub.com/ClickHouse/clickhouse-go/pull/1166)

#### New Contributors

- [@&#8203;omurbekjk](https://togithub.com/omurbekjk) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1166](https://togithub.com/ClickHouse/clickhouse-go/pull/1166)
- [@&#8203;hongker](https://togithub.com/hongker) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1181](https://togithub.com/ClickHouse/clickhouse-go/pull/1181)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.17.1...v2.18.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any
time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNzMuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIzOC4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <[email protected]>
DougManton referenced this pull request in DougManton/opentelemetry-collector-contrib Mar 13, 2024
@renovate @opentelemetrybot @DougManton
fix(deps): update module github.com/clickhouse/clickhouse-go/v2 to v2…
f332e96 
….21.1 (open-telemetry#31065)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/ClickHouse/clickhouse-go/v2](https://togithub.com/ClickHouse/clickhouse-go)
| `v2.17.1` -> `v2.21.1` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.17.1/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.17.1/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>ClickHouse/clickhouse-go
(github.com/ClickHouse/clickhouse-go/v2)</summary>

###
[`v2.21.1`](https://togithub.com/ClickHouse/clickhouse-go/releases/tag/v2.21.1)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.21.0...v2.21.1)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

##### Fixes 🐛

- Adds the ability to Append a zero valued time.Time by
[@&open-telemetry#8203;KevinJoiner](https://togithub.com/KevinJoiner) in
[https://github.com/ClickHouse/clickhouse-go/pull/1228](https://togithub.com/ClickHouse/clickhouse-go/pull/1228)

#### New Contributors

- [@&open-telemetry#8203;KevinJoiner](https://togithub.com/KevinJoiner) made their
first contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1228](https://togithub.com/ClickHouse/clickhouse-go/pull/1228)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.21.0...v2.21.1

###
[`v2.21.0`](https://togithub.com/ClickHouse/clickhouse-go/releases/tag/v2.21.0)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.20.0...v2.21.0)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

##### Enhancements 🎉

- Stream HTTP response body read for decompression by
[@&open-telemetry#8203;rogeryk](https://togithub.com/rogeryk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1213](https://togithub.com/ClickHouse/clickhouse-go/pull/1213)

##### Fixes 🐛

- Fix Date32 extremes values handling by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1218](https://togithub.com/ClickHouse/clickhouse-go/pull/1218)
- Fix column with double quotes PrepareBatch
failed.([#&open-telemetry#8203;1216](https://togithub.com/ClickHouse/clickhouse-go/issues/1216))
by [@&open-telemetry#8203;YenchangChan](https://togithub.com/YenchangChan) in
[https://github.com/ClickHouse/clickhouse-go/pull/1217](https://togithub.com/ClickHouse/clickhouse-go/pull/1217)
- Fix an obscure error in HTTP protocol with LZ4 compression by
[@&open-telemetry#8203;YenchangChan](https://togithub.com/YenchangChan) in
[https://github.com/ClickHouse/clickhouse-go/pull/1230](https://togithub.com/ClickHouse/clickhouse-go/pull/1230)

##### Other Changes 🛠

- Replace deprecated `io/ioutil` functions with equivalents by
[@&open-telemetry#8203;zaneli](https://togithub.com/zaneli) in
[https://github.com/ClickHouse/clickhouse-go/pull/1211](https://togithub.com/ClickHouse/clickhouse-go/pull/1211)
- Run cloud workflow on main branch only by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1219](https://togithub.com/ClickHouse/clickhouse-go/pull/1219)
- ClickHouse 24.2 by [@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1220](https://togithub.com/ClickHouse/clickhouse-go/pull/1220)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.20.0...v2.21.0

###
[`v2.20.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2200-2024-02-28----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.19.0...v2.20.0)

#### What's Changed

##### Enhancements 🎉

- Support \[n]byte/\[]byte type Scan/Append to FixedString column by
[@&open-telemetry#8203;rogeryk](https://togithub.com/rogeryk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1205](https://togithub.com/ClickHouse/clickhouse-go/pull/1205)

##### Other Changes 🛠

- Enable cloud tests by [@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik)
in
[https://github.com/ClickHouse/clickhouse-go/pull/1202](https://togithub.com/ClickHouse/clickhouse-go/pull/1202)
- Removed LowCardinality(UInt64) tests that caused
allow_suspicious_low_cardinality_types related error by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1206](https://togithub.com/ClickHouse/clickhouse-go/pull/1206)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.19.0...v2.20.0

###
[`v2.19.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2190-2024-02-26----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.18.0...v2.19.0)

#### What's Changed

##### Enhancements 🎉

- handle ctx.Done() in acquire by
[@&open-telemetry#8203;threadedstream](https://togithub.com/threadedstream) in
[https://github.com/ClickHouse/clickhouse-go/pull/1199](https://togithub.com/ClickHouse/clickhouse-go/pull/1199)

##### Fixes 🐛

- Fix panic on format nil \*fmt.Stringer type value by
[@&open-telemetry#8203;zaneli](https://togithub.com/zaneli) in
[https://github.com/ClickHouse/clickhouse-go/pull/1200](https://togithub.com/ClickHouse/clickhouse-go/pull/1200)

##### Other Changes 🛠

- Update Go/ClickHouse versions by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1201](https://togithub.com/ClickHouse/clickhouse-go/pull/1201)

#### New Contributors

- [@&open-telemetry#8203;threadedstream](https://togithub.com/threadedstream) made
their first contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1199](https://togithub.com/ClickHouse/clickhouse-go/pull/1199)
- [@&open-telemetry#8203;zaneli](https://togithub.com/zaneli) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1200](https://togithub.com/ClickHouse/clickhouse-go/pull/1200)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.18.0...v2.19.0

###
[`v2.18.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2180-2024-02-01----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.17.1...v2.18.0)

#### What's Changed

##### Enhancements 🎉

- Add WithAllocBufferColStrProvider string column allocator for batch
insert performance boost by
[@&open-telemetry#8203;hongker](https://togithub.com/hongker) in
[https://github.com/ClickHouse/clickhouse-go/pull/1181](https://togithub.com/ClickHouse/clickhouse-go/pull/1181)

##### Fixes 🐛

- Fix bind for seconds scale DateTime by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1184](https://togithub.com/ClickHouse/clickhouse-go/pull/1184)

##### Other Changes 🛠

- resolves
[#&open-telemetry#8203;1163](https://togithub.com/ClickHouse/clickhouse-go/issues/1163)
debugF function is not respected by
[@&open-telemetry#8203;omurbekjk](https://togithub.com/omurbekjk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1166](https://togithub.com/ClickHouse/clickhouse-go/pull/1166)

#### New Contributors

- [@&open-telemetry#8203;omurbekjk](https://togithub.com/omurbekjk) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1166](https://togithub.com/ClickHouse/clickhouse-go/pull/1166)
- [@&open-telemetry#8203;hongker](https://togithub.com/hongker) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1181](https://togithub.com/ClickHouse/clickhouse-go/pull/1181)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.17.1...v2.18.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any
time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNzMuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIzOC4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <[email protected]>
XinRanZhAWS referenced this pull request in XinRanZhAWS/opentelemetry-collector-contrib Mar 13, 2024
@renovate @opentelemetrybot @XinRanZhAWS
fix(deps): update module github.com/clickhouse/clickhouse-go/v2 to v2…
c5e646e 
….21.1 (open-telemetry#31065)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/ClickHouse/clickhouse-go/v2](https://togithub.com/ClickHouse/clickhouse-go)
| `v2.17.1` -> `v2.21.1` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.17.1/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fClickHouse%2fclickhouse-go%2fv2/v2.17.1/v2.21.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>ClickHouse/clickhouse-go
(github.com/ClickHouse/clickhouse-go/v2)</summary>

###
[`v2.21.1`](https://togithub.com/ClickHouse/clickhouse-go/releases/tag/v2.21.1)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.21.0...v2.21.1)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

##### Fixes 🐛

- Adds the ability to Append a zero valued time.Time by
[@&open-telemetry#8203;KevinJoiner](https://togithub.com/KevinJoiner) in
[https://github.com/ClickHouse/clickhouse-go/pull/1228](https://togithub.com/ClickHouse/clickhouse-go/pull/1228)

#### New Contributors

- [@&open-telemetry#8203;KevinJoiner](https://togithub.com/KevinJoiner) made their
first contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1228](https://togithub.com/ClickHouse/clickhouse-go/pull/1228)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.21.0...v2.21.1

###
[`v2.21.0`](https://togithub.com/ClickHouse/clickhouse-go/releases/tag/v2.21.0)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.20.0...v2.21.0)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

##### Enhancements 🎉

- Stream HTTP response body read for decompression by
[@&open-telemetry#8203;rogeryk](https://togithub.com/rogeryk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1213](https://togithub.com/ClickHouse/clickhouse-go/pull/1213)

##### Fixes 🐛

- Fix Date32 extremes values handling by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1218](https://togithub.com/ClickHouse/clickhouse-go/pull/1218)
- Fix column with double quotes PrepareBatch
failed.([#&open-telemetry#8203;1216](https://togithub.com/ClickHouse/clickhouse-go/issues/1216))
by [@&open-telemetry#8203;YenchangChan](https://togithub.com/YenchangChan) in
[https://github.com/ClickHouse/clickhouse-go/pull/1217](https://togithub.com/ClickHouse/clickhouse-go/pull/1217)
- Fix an obscure error in HTTP protocol with LZ4 compression by
[@&open-telemetry#8203;YenchangChan](https://togithub.com/YenchangChan) in
[https://github.com/ClickHouse/clickhouse-go/pull/1230](https://togithub.com/ClickHouse/clickhouse-go/pull/1230)

##### Other Changes 🛠

- Replace deprecated `io/ioutil` functions with equivalents by
[@&open-telemetry#8203;zaneli](https://togithub.com/zaneli) in
[https://github.com/ClickHouse/clickhouse-go/pull/1211](https://togithub.com/ClickHouse/clickhouse-go/pull/1211)
- Run cloud workflow on main branch only by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1219](https://togithub.com/ClickHouse/clickhouse-go/pull/1219)
- ClickHouse 24.2 by [@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1220](https://togithub.com/ClickHouse/clickhouse-go/pull/1220)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.20.0...v2.21.0

###
[`v2.20.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2200-2024-02-28----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.19.0...v2.20.0)

#### What's Changed

##### Enhancements 🎉

- Support \[n]byte/\[]byte type Scan/Append to FixedString column by
[@&open-telemetry#8203;rogeryk](https://togithub.com/rogeryk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1205](https://togithub.com/ClickHouse/clickhouse-go/pull/1205)

##### Other Changes 🛠

- Enable cloud tests by [@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik)
in
[https://github.com/ClickHouse/clickhouse-go/pull/1202](https://togithub.com/ClickHouse/clickhouse-go/pull/1202)
- Removed LowCardinality(UInt64) tests that caused
allow_suspicious_low_cardinality_types related error by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1206](https://togithub.com/ClickHouse/clickhouse-go/pull/1206)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.19.0...v2.20.0

###
[`v2.19.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2190-2024-02-26----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.18.0...v2.19.0)

#### What's Changed

##### Enhancements 🎉

- handle ctx.Done() in acquire by
[@&open-telemetry#8203;threadedstream](https://togithub.com/threadedstream) in
[https://github.com/ClickHouse/clickhouse-go/pull/1199](https://togithub.com/ClickHouse/clickhouse-go/pull/1199)

##### Fixes 🐛

- Fix panic on format nil \*fmt.Stringer type value by
[@&open-telemetry#8203;zaneli](https://togithub.com/zaneli) in
[https://github.com/ClickHouse/clickhouse-go/pull/1200](https://togithub.com/ClickHouse/clickhouse-go/pull/1200)

##### Other Changes 🛠

- Update Go/ClickHouse versions by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1201](https://togithub.com/ClickHouse/clickhouse-go/pull/1201)

#### New Contributors

- [@&open-telemetry#8203;threadedstream](https://togithub.com/threadedstream) made
their first contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1199](https://togithub.com/ClickHouse/clickhouse-go/pull/1199)
- [@&open-telemetry#8203;zaneli](https://togithub.com/zaneli) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1200](https://togithub.com/ClickHouse/clickhouse-go/pull/1200)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.18.0...v2.19.0

###
[`v2.18.0`](https://togithub.com/ClickHouse/clickhouse-go/blob/HEAD/CHANGELOG.md#v2180-2024-02-01----Release-notes-generated-using-configuration-in-githubreleaseyml-at-main---)

[Compare
Source](https://togithub.com/ClickHouse/clickhouse-go/compare/v2.17.1...v2.18.0)

#### What's Changed

##### Enhancements 🎉

- Add WithAllocBufferColStrProvider string column allocator for batch
insert performance boost by
[@&open-telemetry#8203;hongker](https://togithub.com/hongker) in
[https://github.com/ClickHouse/clickhouse-go/pull/1181](https://togithub.com/ClickHouse/clickhouse-go/pull/1181)

##### Fixes 🐛

- Fix bind for seconds scale DateTime by
[@&open-telemetry#8203;jkaflik](https://togithub.com/jkaflik) in
[https://github.com/ClickHouse/clickhouse-go/pull/1184](https://togithub.com/ClickHouse/clickhouse-go/pull/1184)

##### Other Changes 🛠

- resolves
[#&open-telemetry#8203;1163](https://togithub.com/ClickHouse/clickhouse-go/issues/1163)
debugF function is not respected by
[@&open-telemetry#8203;omurbekjk](https://togithub.com/omurbekjk) in
[https://github.com/ClickHouse/clickhouse-go/pull/1166](https://togithub.com/ClickHouse/clickhouse-go/pull/1166)

#### New Contributors

- [@&open-telemetry#8203;omurbekjk](https://togithub.com/omurbekjk) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1166](https://togithub.com/ClickHouse/clickhouse-go/pull/1166)
- [@&open-telemetry#8203;hongker](https://togithub.com/hongker) made their first
contribution in
[https://github.com/ClickHouse/clickhouse-go/pull/1181](https://togithub.com/ClickHouse/clickhouse-go/pull/1181)

**Full Changelog**:
ClickHouse/clickhouse-go@v2.17.1...v2.18.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any
time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNzMuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIzOC4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <[email protected]>
@jkaflik jkaflik mentioned this pull request Mar 28, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jkaflik jkaflik jkaflik approved these changes

Assignees

@jkaflik jkaflik

Labels
bug fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@KevinJoiner @CLAassistant @jkaflik