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

murron logger #107

Open
wants to merge 145 commits into
base: master
Choose a base branch
from
Open

murron logger #107

wants to merge 145 commits into from

Conversation

demmer
Copy link
Collaborator

@demmer demmer commented Aug 30, 2018

WIP to add a slack-specific plugin that writes logs directly to murron.

Still untested.

demmer and others added 30 commits January 27, 2018 08:31
Modeling after the vtgate mysql server implementation, add a simple
module that implements the mysql server protocol handlers and
proxies all requests to the appropriate methods in a queryservice
backend.

This includes optional support for query normalization.
Add a binary to wrap a standalone queryservice with a grpc service
interface and a mysql server implementation using the mysqlproxy
module to direct to the queryservice.
Following the example of vtgate, add support to set autocommit on
or off in the session, update the execution path to use ExecuteBatch
for single round-trip autocommit semantics, and add tests to ensure
the behavior matches mysql.
This allows EXPLAIN and other such statements to go through without
query normalization.
Co-authored-by: Michael Demmer <mdemmer@slack-corp.com>
…8-02-07"

This reverts commit 713273a, reversing
changes made to edc17cd.
…8-02-07"

This reverts commit 0ce5f56, reversing
changes made to 713273a.
…ray-comment"

This reverts commit 80d72ac, reversing
changes made to 732edb1.
…-29"

This reverts commit d48e84a, reversing
changes made to 83866b3.
rafael and others added 20 commits January 30, 2019 11:12
Patch to handle this case while we move towards a permanent fix upstream.
cf. vitessio#4661 vitessio#4669
Signed-off-by: Rafael Chacon <rafael@slack-corp.com>
* Deprecate the concept of regions.
* Favor aliases instead

Signed-off-by: Rafael Chacon <rafael@slack-corp.com>
Signed-off-by: Rafael Chacon <rafael@slack-corp.com>
It's been busy and exciting in vitess upstream. Some changes that felt worth calling out as they could impact assumptions about behavior:

- vitessio#4832 — this adds a timeout that is 3x the `healthCheckInterval` which at which point the tablet will stop reporting as healthy
- vitessio#4814 — this changes when command line args are used by vttablet
- vitessio#4811 — a planned reparent now sets `super_read_only`
- vitessio#4805 — behavior coming back from a failed backup is different

**Slack changes**  
        @demmer vitessio#4827 Correct suppress logging for begin...commit in autocommit
        @rafael vitessio#4824 adds timeouts for all statements
        @demmer vitessio#4826 adds logging of stack traces as opt-in
        @demmer vitessio#4819 remove begin/commit logs from autocommit txns
        @demmer vitessio#4796 improves support for vtexplain for begin/dml/dml/commit txns

**Non slack changes**  
        vitessio#4839 Improve behavior with reference table routing & vreplication
        vitessio#4833 Support query routing given there could be multiple targets for a table
        vitessio#4832 tablets get new health check behavior (health checks time out)
        vitessio#4837 refresh bug around row streamer handling creds
        vitessio#4830 apply a default value to the db_name attribute in local_metadata
        vitessio#4785 SHOW SCHEMAS aliased to SHOW DATABASES
        vitessio#4829 add experimental support for split clone & vertical split clone via vreplication
        vitessio#4822 do not normalize in order by
        vitessio#4791 Adds vreplication row streaming support
        vitessio#4814 command line flags now used regardless of management mode
        vitessio#4811 vttablet sets super_read_only during planned reparent
        vitessio#4803 xtrabackup testing
        vitessio#4727 support multiple vttablets running against one mysql instance
        vitessio#4746 TopoCat can now produce JSON output
        vitessio#4805 mysqld is now restarted after a failed backup
        vitessio#4685 macos bootstrap bug
        vitessio#4874 ZK opts and java land
        vitessio#4695 Adds support for xtrabackup
        vitessio#4794 fixes build failures
        vitessio#4725 changes mysql8.0 start args
        vitessio#4736 introduce new states to support vreplication
        vitessio#4788 expands orc error

**Docs etc**  
        vitessio#4831 lint
        vitessio#4827 docs
        vitessio#4816 const declaration cleanup
        vitessio#4820 const declaration cleanup
        vitessio#4825 docs
        vitessio#4818 docs
        vitessio#4809 docs
        vitessio#4812 moves consts around
        vitessio#4813 docs
        vitessio#4808 docs
        vitessio#4800 docs
        vitessio#4795 docs
Signed-off-by: Michael Demmer <mdemmer@slack-corp.com>
Signed-off-by: Michael Demmer <mdemmer@slack-corp.com>
Signed-off-by: Michael Demmer <mdemmer@slack-corp.com>
Signed-off-by: Michael Demmer <mdemmer@slack-corp.com>
Signed-off-by: Michael Demmer <mdemmer@slack-corp.com>
Signed-off-by: Michael Demmer <mdemmer@slack-corp.com>
@demmer
Copy link
Collaborator Author

demmer commented Jun 7, 2019

For fun I just updated this PR to the latest versions of murron and now it won't actually build properly....
Someone needs to spend some time in govendor fun hell getting things to sort out.

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.

7 participants