Skip to content

Add callerid support for txlogz and querylogz#1198

Merged
guokeno0 merged 9 commits intovitessio:masterfrom
guokeno0:logz
Oct 11, 2015
Merged

Add callerid support for txlogz and querylogz#1198
guokeno0 merged 9 commits intovitessio:masterfrom
guokeno0:logz

Conversation

@guokeno0
Copy link
Copy Markdown
Contributor

No description provided.

@guokeno0
Copy link
Copy Markdown
Contributor Author

@sougou

This is the back-port from internal commit

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should be context.Background(). Apparently TODO is to be used only for non-text code.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

@sougou
Copy link
Copy Markdown
Contributor

sougou commented Oct 11, 2015

lgtm. one comment.

guokeno0 added a commit that referenced this pull request Oct 11, 2015
Add callerid support for txlogz and querylogz
@guokeno0 guokeno0 merged commit a1d029f into vitessio:master Oct 11, 2015
@guokeno0 guokeno0 deleted the logz branch October 11, 2015 06:43
notfelineit pushed a commit to planetscale/vitess that referenced this pull request Nov 16, 2022
…#11478) (vitessio#1198)

* Fix check_make_sizegen

Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>

* Apply review suggestion

Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>

* Fail the static_check_etc workflow when a script fails

Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>

* Remove test code

Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
Signed-off-by: Max Englander <max@planetscale.com>
Co-authored-by: vitess-bot[bot] <108069721+vitess-bot[bot]@users.noreply.github.com>
Co-authored-by: Florent Poinsard <florent.poinsard@outlook.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants