Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

fix pass []any as any in logQueryStats #1480

Merged
merged 1 commit into from
Jul 28, 2022

Conversation

alingse
Copy link
Contributor

@alingse alingse commented Jul 10, 2022

Description

fix pass []any as any in Next, call logQueryStats need with args...

and fix logQueryStats fmt.Sprintf("%v", args...) if args has many this will not format as expected.

see github actions result here https://github.com/alingse/asasalint/runs/7270916179?check_suite_focus=true

Merge requirements

Please take into account the following non-code changes that you may need to make with your PR:

  • CHANGELOG entry for user-facing changes
  • Updated the relevant documentation

and fix logQueryStats `fmt.Sprintf("%v", args...)` if args has many this will not format as expected.
@alingse alingse requested a review from a team as a code owner July 10, 2022 13:41
@CLAassistant
Copy link

CLAassistant commented Jul 10, 2022

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@niksajakovljevic
Copy link
Contributor

@alingse Thanks for contributing. To merge this PR we need you to sign CLA. Thanks

@niksajakovljevic niksajakovljevic merged commit 6e0be57 into timescale:master Jul 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants