Skip to content

[Feature]: Upgrade Memory Storage to v2 API #7052

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
Open
6 of 8 tasks
Manik2708 opened this issue Apr 19, 2025 · 0 comments
Open
6 of 8 tasks

[Feature]: Upgrade Memory Storage to v2 API #7052

Manik2708 opened this issue Apr 19, 2025 · 0 comments
Labels
area/storage enhancement v2

Comments

@Manik2708
Copy link
Contributor

Manik2708 commented Apr 19, 2025 •
edited
Loading

Requirement

Requirement

  • Implement WriteTraces
  • Upgrade GetServices and GetOperations
  • Upgrade FindTraceIDs
  • Upgrade GetTraces
  • Upgrade FindTraces
  • Upgrade DependencyStore
  • Refactor v1 Factory to make it reusable for v2
  • Upgrade Memory Factory to use v2 APIs

Problem

Part of #6458

Proposal

No response

Open questions

No response
@Manik2708 Manik2708 mentioned this issue Apr 19, 2025
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this issue Apr 19, 2025
@Manik2708
[v2] Implement GetOperations and GetServices for memory backend (#…
2f7563c 
…7053)

## Which problem is this PR solving?
- Fixes a part of: #7052 

## Description of the changes
- Implement GetOperations and GetServices for memory backend

## How was this change tested?
- Unit tests

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

Signed-off-by: Manik2708 <mehtamanik96@gmail.com>
@Manik2708 Manik2708 mentioned this issue Apr 21, 2025
Merged
4 tasks
@yurishkuro yurishkuro moved this to In Progress in Jaeger V2 May 3, 2025
amilbcahat pushed a commit to amilbcahat/jaeger that referenced this issue May 4, 2025
@Manik2708 @amol-verma-allen
[v2] Implement GetOperations and GetServices for memory backend (j…
e57dc53 
…aegertracing#7053)

## Which problem is this PR solving?
- Fixes a part of: jaegertracing#7052 

## Description of the changes
- Implement GetOperations and GetServices for memory backend

## How was this change tested?
- Unit tests

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

Signed-off-by: Manik2708 <mehtamanik96@gmail.com>
amilbcahat pushed a commit to amilbcahat/jaeger that referenced this issue May 4, 2025
@Manik2708 @amol-verma-allen
[v2] Implement GetOperations and GetServices for memory backend (j…
057c896 
…aegertracing#7053)

## Which problem is this PR solving?
- Fixes a part of: jaegertracing#7052

## Description of the changes
- Implement GetOperations and GetServices for memory backend

## How was this change tested?
- Unit tests

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

Signed-off-by: Manik2708 <mehtamanik96@gmail.com>
Signed-off-by: amol-verma-allen <amol.verma@allen.in>
github-merge-queue bot pushed a commit that referenced this issue May 18, 2025
@Manik2708 @yurishkuro
[v2] Implement FindTraces for memory backend (#7062)
1be2293 
## Which problem is this PR solving?
- Fixes a part of: #7052 

## Description of the changes
- Implement `FindTraces` for memory backend

## How was this change tested?
- Unit Tests

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

---------

Signed-off-by: Manik2708 <mehtamanik96@gmail.com>
Signed-off-by: Yuri Shkuro <github@ysh.us>
Co-authored-by: Yuri Shkuro <github@ysh.us>
@Manik2708 Manik2708 mentioned this issue May 18, 2025
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this issue May 20, 2025
@Manik2708
[v2] Implement FindTraceIDs for memory backend (#7143)
c1fb73a 
## Which problem is this PR solving?
- Fixes a part of: #7052 

## Description of the changes
- Implement FindTraceIds for memoty backend

## How was this change tested?
- Unit Tests

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

---------

Signed-off-by: Manik2708 <mehtamanik96@gmail.com>
@Manik2708 Manik2708 mentioned this issue May 21, 2025
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this issue May 21, 2025
@Manik2708
[v2] Implement GetTraces for memory backend (#7152)
b12345b 
## Which problem is this PR solving?
- Fixes a part of: #7052 

## Description of the changes
- Implement `GetTraces` for memory backend.

## How was this change tested?
- Unit Tests

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

---------

Signed-off-by: Manik2708 <mehtamanik96@gmail.com>
@Manik2708 Manik2708 mentioned this issue May 23, 2025
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this issue May 24, 2025
@Manik2708
[v2] Implement GetDependencies for memory backend (#7154)
463d928 
## Which problem is this PR solving?
- Fixes a part of: #7052 

## Description of the changes
- Implement `GetDependencies` for memory backend

## How was this change tested?
- Unit Tests

## Checklist
- [x] I have read
https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
- [x] I have signed all commits
- [x] I have added unit tests for the new functionality
- [x] I have run lint and test steps successfully
  - for `jaeger`: `make lint test`
  - for `jaeger-ui`: `npm run lint` and `npm run test`

---------

Signed-off-by: Manik2708 <mehtamanik96@gmail.com>
@Manik2708 Manik2708 mentioned this issue May 24, 2025
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/storage enhancement v2
Projects
Jaeger V2
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Manik2708