Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CQLRpcServerEnv class should be removed #410

Closed
robertpang opened this issue Jul 30, 2018 · 1 comment
Closed

CQLRpcServerEnv class should be removed #410

robertpang opened this issue Jul 30, 2018 · 1 comment
Assignees
@robertpang
Labels
kind/enhancement This is an enhancement of an existing feature
Projects
YCQL

Comments

@robertpang
Copy link
Contributor

The class CQLRpcServerEnv was used to implement system.local virtual table locally. The system table has since then been moved to yb-master so the class should be removed.
@robertpang robertpang added the kind/enhancement This is an enhancement of an existing feature label Jul 30, 2018
@robertpang robertpang self-assigned this Jul 30, 2018
@robertpang robertpang added this to To Do in YCQL via automation Jul 30, 2018
@robertpang robertpang moved this from To Do to In progress in YCQL Jul 30, 2018
mbautin pushed a commit that referenced this issue Aug 1, 2018
@robertpang
#410: Remove unused CQLRpcServerEnv
9bc7dc1 
Summary: The class CQLRpcServerEnv was used to implement system.local virtual table locally in YCQL proxy in yb-tserver. The system table has since then been moved to yb-master so the class should be removed.

Test Plan: Jenkins

Reviewers: mihnea

Reviewed By: mihnea

Subscribers: yql

Differential Revision: https://phabricator.dev.yugabyte.com/D5238
@robertpang
Copy link
Contributor Author

Fixed in commit 9bc7dc1.

YCQL automation moved this from In progress to Done Aug 1, 2018
@ryan-ally ryan-ally mentioned this issue Dec 21, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@robertpang robertpang

Labels
kind/enhancement This is an enhancement of an existing feature
Projects
YCQL
  
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@robertpang