-
Notifications
You must be signed in to change notification settings - Fork 68
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
WriterMetrics and config files dependency redirection #450
WriterMetrics and config files dependency redirection #450
Conversation
Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com>
Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com>
Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com>
Codecov Report
@@ Coverage Diff @@
## main #450 +/- ##
============================================
+ Coverage 72.13% 72.20% +0.06%
Complexity 375 375
============================================
Files 44 44
Lines 2591 2590 -1
Branches 173 173
============================================
+ Hits 1869 1870 +1
+ Misses 614 612 -2
Partials 108 108
|
@@ -299,7 +299,7 @@ dependencies { | |||
implementation 'org.bouncycastle:bcprov-jdk15on:1.70' | |||
implementation 'org.bouncycastle:bcpkix-jdk15on:1.70' | |||
implementation "org.opensearch:performanceanalyzer-rca:${version}" | |||
implementation "org.opensearch:performanceanalyzer-commons:${version}" | |||
implementation "org.opensearch:performance-analyzer-commons:1.0.0-SNAPSHOT" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.0.0-SNAPSHOT
is meant for tracking active development and we use the release version otherwise.
Let us fix this in follow-up PR.
* Config files dependency redirection. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> * WriterMetrics dependency redirection. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> * Commons dependency, license and notice. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> --------- Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> (cherry picked from commit bd5bc40)
* Config files dependency redirection. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> * WriterMetrics dependency redirection. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> * Commons dependency, license and notice. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> --------- Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> (cherry picked from commit bd5bc40)
* Config files dependency redirection. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> * WriterMetrics dependency redirection. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> * Commons dependency, license and notice. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> --------- Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> (cherry picked from commit bd5bc40) Co-authored-by: Filip Drobnjaković <drobnjakovicfilip@gmail.com>
* Config files dependency redirection. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> * WriterMetrics dependency redirection. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> * Commons dependency, license and notice. Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> --------- Signed-off-by: Filip Drobnjakovic <drobnjakovicfilip@gmail.com> (cherry picked from commit bd5bc40) Co-authored-by: Filip Drobnjaković <drobnjakovicfilip@gmail.com>
Redirected
WriterMetrics
and config file dependencies from PA-RCA repo to Commons.Additional context
Add any other context or screenshots about the feature request here.
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.