8000 [Feature]: Upgrade Elasticsearch Storage to v2 API · Issue #7034 · jaegertracing/jaeger · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[Feature]: Upgrade Elasticsearch Storage to v2 API #7034

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 15, 2025 · 0 comments
Open
6 of 8 tasks

[Feature]: Upgrade Elasticsearch Storage to v2 API #7034

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

Comments

@Manik2708
Copy link
Contributor
Manik2708 commented Apr 15, 2025 •
edited
Loading

Requirement

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

Problem

Part of #6458
@Manik2708 Manik2708 mentioned this issue Apr 15, 2025
Merged
4 tasks
@yurishkuro yurishkuro changed the title [Feature]: Upgrade Backend Storage APIs to v2 for ElasticSearch [Feature]: Upgrade Elasticsearch Storage to v2 API Apr 15, 2025
This was referenced Apr 16, 2025
Open
Merged
github-merge-queue bot pushed a commit that referenced this issue Apr 18, 2025
@Manik2708 @yurishkuro
[ES] Refactor DependencyStore to make it reusable for v2 APIs (#7044)
600309a 
## Which problem is this PR solving?
- Fixes a part of #7034 

## Description of the changes
- Refactoring of Dependnency Store to make it resuable for v2 APIs

## 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 <yurishkuro@users.noreply.github.com>
Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
github-merge-queue bot pushed a commit that referenced this issue Apr 19, 2025
@Manik2708
[v2] Implement FindTraceIDs for ES/OS (#7035)
4f46ecd 
## Which problem is this PR solving?
- Fixes a part of: #7034 

## Description of the changes
- Implement `FindTraceIDs` for ES/OS 

## 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 19, 2025
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this issue Apr 20, 2025
@Manik2708
[ES][v2] Implement GetTraces for ES/OS (#7054)
8e0e9a7 
## Which problem is this PR solving?
- Fixes a part of: #7034 

## Description of the changes
- Implement `GetTraces` for ES/OS

## 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>
@yurishkuro yurishkuro mentioned this issue Apr 20, 2025
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this issue Apr 20, 2025
@Manik2708
[ES][v2] Implement FindTraces for ES/OS for v2 (#7021)
8e81bf8 
## Which problem is this PR solving?
- Fixes a part of: #7034

## Description of the changes
- Implement the `FindTraces` for ES/OS for v2

## 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>
This was referenced Apr 23, 2025
Merged
Closed
Merged
github-merge-queue bot pushed a commit that referenced this issue Apr 27, 2025
@Manik2708
[ES] Make NestedTags and ElevatedTags distinction at `CoreSpanRea…
c3f8f21 
…der` level (#7067)

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

## Description of the changes
- Make `NestedTags` and `ElevatedTags` distinction at `CoreSpanReader`
level and a follow-up PR for #6946

## How was this change tested?
- Unit And Integration 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>
github-merge-queue bot pushed a commit that referenced this issue Apr 27, 2025
@Manik2708
[ES][v2] Move CoreDependencyStore and dbmodel from v1 to v2 (#7079)
9258138 
## Which problem is this PR solving?
- Fixes a part of: #7034 

## Description of the changes
- The shared code need to be displaced completely to v2. This is a
pre-requisite PR for implementing the dependency store in v2

## 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 29, 2025
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this issue Apr 29, 2025
@Manik2708
[ES][v2] Implement GetDependenies and WriteDependencies (#7085)
09617d1 
## Which problem is this PR solving?
- Fixes a part of: #7034 

## Description of the changes
- Upgrade DependencyStore for ES

## 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 30, 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 storage/elasticsearch v2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Manik2708
0