sql icon indicating copy to clipboard operation
sql copied to clipboard

Prometheus Connector Implementation

Open vamsi-amazon opened this issue 2 years ago • 1 comments

Signed-off-by: vamsi-amazon [email protected]

Description

This CR covers implementation of catalog and prometheus connector.

Things not covered.

  1. Integ tests and doc tests.
  2. Memory protector implementation.
  3. Basic authentication support for prometheus. Will update the list during the course if review and will create a separate PR.

Issues Resolved

https://github.com/opensearch-project/sql/issues/561

Check List

  • [ ] New functionality includes testing.
    • [ ] All tests pass, including unit test, integration test and doctest
  • [ ] New functionality has been documented.
    • [ ] New functionality has javadoc added
    • [ ] New functionality has user manual doc added
  • [ ] Commits are signed per the DCO using --signoff

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.

vamsi-amazon avatar Jul 19 '22 05:07 vamsi-amazon

Codecov Report

Merging #700 (ce9e87f) into main (3f545f2) will increase coverage by 0.04%. The diff coverage is 99.12%.

@@             Coverage Diff              @@
##               main     #700      +/-   ##
============================================
+ Coverage     94.74%   94.79%   +0.04%     
- Complexity     2857     2879      +22     
============================================
  Files           283      285       +2     
  Lines          7676     7769      +93     
  Branches        561      566       +5     
============================================
+ Hits           7273     7365      +92     
- Misses          349      350       +1     
  Partials         54       54              
Flag Coverage Δ
query-workbench 62.76% <ø> (ø)
sql-engine 97.76% <99.12%> (+0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...a/org/opensearch/sql/analysis/TypeEnvironment.java 100.00% <ø> (ø)
...org/opensearch/sql/expression/NamedExpression.java 100.00% <ø> (ø)
.../main/java/org/opensearch/sql/utils/Constants.java 0.00% <0.00%> (ø)
...ain/java/org/opensearch/sql/analysis/Analyzer.java 100.00% <100.00%> (ø)
...org/opensearch/sql/planner/DefaultImplementor.java 100.00% <100.00%> (ø)
.../main/java/org/opensearch/sql/planner/Planner.java 100.00% <100.00%> (ø)
...search/sql/planner/logical/LogicalNativeQuery.java 100.00% <100.00%> (ø)
...opensearch/sql/planner/logical/LogicalPlanDSL.java 100.00% <100.00%> (ø)
...ch/sql/planner/logical/LogicalPlanNodeVisitor.java 100.00% <100.00%> (ø)
...pensearch/sql/planner/logical/LogicalRelation.java 100.00% <100.00%> (ø)
... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

codecov-commenter avatar Jul 19 '22 06:07 codecov-commenter