Skip to content

[Feature Branch] Add constants and specify types#1144

Merged
cliu123 merged 1 commit intoopensearch-project:2.x_aggregation_viewfrom
cliu123:2.x_namespace-as-tenant
Oct 17, 2022
Merged

[Feature Branch] Add constants and specify types#1144
cliu123 merged 1 commit intoopensearch-project:2.x_aggregation_viewfrom
cliu123:2.x_namespace-as-tenant

Conversation

@cliu123
Copy link
Copy Markdown
Member

@cliu123 cliu123 commented Oct 17, 2022

Signed-off-by: Chang Liu lc12251109@gmail.com

Description

[Describe what this change achieves]

Category

[Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation]

Why these changes are required?

What is the old behavior before changes and new behavior after changes?

Issues Resolved

[List any issues this PR will resolve (Is this a backport? If so, please add backport PR # and/or commits #)]

Testing

[Please provide details of testing done: unit testing, integration testing and manual testing]

Check List

  • New functionality includes testing
  • New functionality has been documented
  • 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.

Signed-off-by: Chang Liu <lc12251109@gmail.com>
@cliu123 cliu123 force-pushed the 2.x_namespace-as-tenant branch from d8a91f1 to b28e780 Compare October 17, 2022 22:15
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #1144 (b28e780) into 2.x_aggregation_view (6a823da) will not change coverage.
The diff coverage is n/a.

@@                  Coverage Diff                  @@
##           2.x_aggregation_view    #1144   +/-   ##
=====================================================
  Coverage                 74.29%   74.29%           
=====================================================
  Files                        86       86           
  Lines                      1879     1879           
  Branches                    246      246           
=====================================================
  Hits                       1396     1396           
  Misses                      426      426           
  Partials                     57       57           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@cliu123 cliu123 marked this pull request as ready for review October 17, 2022 22:50
@cliu123 cliu123 requested a review from a team October 17, 2022 22:50
@cliu123 cliu123 merged commit 736a1d9 into opensearch-project:2.x_aggregation_view Oct 17, 2022
@cliu123 cliu123 deleted the 2.x_namespace-as-tenant branch October 17, 2022 22:51
@cliu123 cliu123 restored the 2.x_namespace-as-tenant branch October 17, 2022 22:51
cliu123 added a commit that referenced this pull request Oct 17, 2022
Signed-off-by: Chang Liu <lc12251109@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants