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

[DocDB] Move opid from util to common #21291

Closed
1 task done
es1024 opened this issue Mar 4, 2024 · 0 comments
Closed
1 task done

[DocDB] Move opid from util to common #21291

es1024 opened this issue Mar 4, 2024 · 0 comments
Assignees
@es1024
Labels
area/docdb YugabyteDB core features kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue

Comments

@es1024
Copy link
Contributor

es1024 commented Mar 4, 2024 •
edited by jira bot
Loading

Jira Link: DB-10207

Description

OpId/OpIdPB are currently located in util, and opid_proto has a yb_common_proto dependency.

#21290 will involve usage of OpIdPB in yb_common_proto/OpId in common, so opid.(proto/h/cc) need to be moved from util/ to common/.

Context: #15533

Issue Type

kind/enhancement

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.
@es1024 es1024 added area/docdb YugabyteDB core features status/awaiting-triage Issue awaiting triage labels Mar 4, 2024
@es1024 es1024 self-assigned this Mar 4, 2024
@yugabyte-ci yugabyte-ci added kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue and removed status/awaiting-triage Issue awaiting triage labels Mar 4, 2024
es1024 added a commit that referenced this issue Mar 6, 2024
@es1024
[#21291] docdb: Move opid to common/
91213ad 
Summary:
Move opid.proto/opid.cc/opid.h to common/, needed for D31900.
Jira: DB-10207

Test Plan: Jenkins: compile only

Reviewers: sergei

Reviewed By: sergei

Subscribers: yql, ycdcxcluster, ybase, rthallam, bogdan

Tags: #jenkins-ready

Differential Revision: https://phorge.dev.yugabyte.com/D32632
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@es1024 es1024

Labels
area/docdb YugabyteDB core features kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@es1024 @rthallamko3 @yugabyte-ci