/cockroach

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.

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

[WIP] distsql: add ProcessorGroup #30883

Open
wants to merge 1 commit into
base: master
from

Conversation

Reviewers

@cockroachdb/sql-execution-prs sql-execution-prs

@cockroachdb/distsql-prs distsql-prs

@jordanlewis jordanlewis

@solongordon solongordon

@arjunravinarayan arjunravinarayan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@asubiotto @cockroach-teamcity
@asubiotto
Contributor

asubiotto commented Oct 2, 2018

Currently a WIP and not ready for a full review, but wanted to get feedback on the approach and take a step back.

Refer to this comment as to why I think it's useful to have this. Will be writing a better commit message later on.

@asubiotto asubiotto requested review from jordanlewis, solongordon and arjunravinarayan Oct 2, 2018

@asubiotto asubiotto requested review from cockroachdb/distsql-prs as code owners Oct 2, 2018

@cockroach-teamcity

This comment has been minimized.

Show comment
Hide comment
@cockroach-teamcity

cockroach-teamcity Oct 2, 2018

Member

This change is Reviewable
Member

cockroach-teamcity commented Oct 2, 2018

This change is Reviewable
@asubiotto

This comment has been minimized.

Show comment
Hide comment
@asubiotto

asubiotto Oct 2, 2018

Contributor

cc @RaduBerinde
Contributor

asubiotto commented Oct 2, 2018

cc @RaduBerinde
@asubiotto
distsql: add ProcessorGroup 
Release note: None
Loading status checks…
a73b8a7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment