Skip to content

Performance Improvement for Datetime formats (update version checks to 2.12.0)#10754

Merged
andrross merged 1 commit into
opensearch-project:mainfrom
reta:fix.datetime
Oct 19, 2023
Merged

Performance Improvement for Datetime formats (update version checks to 2.12.0)#10754
andrross merged 1 commit into
opensearch-project:mainfrom
reta:fix.datetime

Conversation

@reta
Copy link
Copy Markdown
Contributor

@reta reta commented Oct 19, 2023 •
edited
Loading

Description

Performance Improvement for Datetime formats (update version checks to 2.12.0)

Related Issues

#10448

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)
  • Public documentation issue/PR created

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.

@reta
Performance Improvement for Datetime formats (update version checks t…
1e8a9a4 
…o 2.12.0)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@reta reta mentioned this pull request Oct 19, 2023
Merged
8 tasks
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 19, 2023

Compatibility status:

Checks if related components are compatible with change 1e8a9a4

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git]

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 19, 2023

Gradle Check (Jenkins) Run Completed with:

@reta
Copy link
Copy Markdown
Contributor Author

reta commented Oct 19, 2023

#10755
#10756

@mch2 mch2 mentioned this pull request Oct 19, 2023
Merged
7 tasks
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 19, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.search.SearchWeightedRoutingIT.testShardRoutingWithNetworkDisruption_FailOpenEnabled
      1 org.opensearch.remotestore.RemoteStoreStatsIT.testNonZeroPrimaryStatsOnNewlyCreatedIndexWithZeroDocs

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 19, 2023

Codecov Report

Merging #10754 (1e8a9a4) into main (69f6f4e) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##               main   #10754   +/-   ##
=========================================
  Coverage     71.11%   71.11%           
+ Complexity    58530    58471   -59     
=========================================
  Files          4854     4854           
  Lines        276049   276049           
  Branches      40168    40168           
=========================================
+ Hits         196311   196318    +7     
+ Misses        63385    63296   -89     
- Partials      16353    16435   +82
Files Coverage Δ
...ain/java/org/opensearch/search/DocValueFormat.java 65.08% <66.66%> (+0.59%) ⬆️

... and 458 files with indirect coverage changes

@andrross andrross merged commit da24ca7 into opensearch-project:main Oct 19, 2023
austintlee pushed a commit to austintlee/OpenSearch that referenced this pull request Oct 23, 2023
@reta @austin-aryn-ai
Performance Improvement for Datetime formats (update version checks t…
c781eef 
…o 2.12.0) (opensearch-project#10754)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
@reta
Performance Improvement for Datetime formats (update version checks t…
82e5b1c 
…o 2.12.0) (opensearch-project#10754)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mch2 mch2 mch2 approved these changes

@andrross andrross andrross approved these changes

@abbashus abbashus Awaiting requested review from abbashus

@anasalkouz anasalkouz Awaiting requested review from anasalkouz

@Bukhtawar Bukhtawar Awaiting requested review from Bukhtawar Bukhtawar is a code owner

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE

@dblock dblock Awaiting requested review from dblock

@dbwiddis dbwiddis Awaiting requested review from dbwiddis dbwiddis is a code owner

@dreamer-89 dreamer-89 Awaiting requested review from dreamer-89

@gbbafna gbbafna Awaiting requested review from gbbafna gbbafna is a code owner

@kartg kartg Awaiting requested review from kartg

@kotwanikunal kotwanikunal Awaiting requested review from kotwanikunal kotwanikunal is a code owner

@msfroh msfroh Awaiting requested review from msfroh msfroh is a code owner

@nknize nknize Awaiting requested review from nknize

@owaiskazi19 owaiskazi19 Awaiting requested review from owaiskazi19 owaiskazi19 is a code owner

@peternied peternied Awaiting requested review from peternied

@Rishikesh1159 Rishikesh1159 Awaiting requested review from Rishikesh1159

@ryanbogan ryanbogan Awaiting requested review from ryanbogan

@sachinpkale sachinpkale Awaiting requested review from sachinpkale sachinpkale is a code owner

@saratvemulapalli saratvemulapalli Awaiting requested review from saratvemulapalli saratvemulapalli is a code owner

@setiah setiah Awaiting requested review from setiah

@shwetathareja shwetathareja Awaiting requested review from shwetathareja shwetathareja is a code owner

@sohami sohami Awaiting requested review from sohami sohami is a code owner

@VachaShah VachaShah Awaiting requested review from VachaShah

+2 more reviewers

@CaptainDredge CaptainDredge CaptainDredge approved these changes

@sarthakaggarwal97 sarthakaggarwal97 sarthakaggarwal97 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

skip-changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@reta @mch2 @andrross @CaptainDredge @sarthakaggarwal97