Skip to content
This repository has been archived by the owner on Aug 24, 2022. It is now read-only.

PMM-9634 Timestamp bucket_start_time for newer versions. #329

Open
wants to merge 6 commits into
base: PMM-9630-status-code-usage
Choose a base branch
from

Conversation

JiriCtvrtka
Copy link
Contributor

PMM-9634

Build: SUBMODULES-0

  • Links to other linked pull requests (optional).

Branch is based on 9630.

@@ -106,7 +106,7 @@ type field struct {
pointer interface{}
}

func NewPgStatMonitorStructs(v pgStatMonitorVersion) (*pgStatMonitor, reform.View) {
func NewPgStatMonitorStructs(v pgStatMonitorVersion, p pgStatMonitorPrerelease) (*pgStatMonitor, reform.View) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci-lint] reported by reviewdog 🐶
Function 'NewPgStatMonitorStructs' is too long (114 > 60) (funlen)

@JiriCtvrtka
Copy link
Contributor Author

2 failed tests are not related to PR. See empty PR failed on same tests here: #328

@JiriCtvrtka JiriCtvrtka marked this pull request as ready for review March 10, 2022 11:57
@JiriCtvrtka JiriCtvrtka requested a review from BupycHuk as a code owner March 10, 2022 11:57
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