Skip to content
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

Fixing various javadocs #9817

Merged
merged 6 commits into from
Sep 10, 2023
Merged

Fixing various javadocs #9817

merged 6 commits into from
Sep 10, 2023

Conversation

ker2x
Copy link
Contributor

@ker2x ker2x commented Sep 6, 2023

Greetings !
I'm in the process of completing javadoc of OpenSearch.
I have other javadoc fix in progress but not yet ready.

I tried my best following the contributing rule. i hope it's ok.
I didn't create a new issue as there is this one already #221

Should i make one large pull request or many small one ?

Thank you,

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

Compatibility status:

Checks if related components are compatible with change 157dd0c

Incompatible components

Incompatible components: [https://github.com/opensearch-project/security.git]

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.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/opensearch-oci-object-storage.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git]

@kotwanikunal
Copy link
Member

@kotwanikunal
Copy link
Member

I fixed the previous error (my bad) But this new failure doesn't appear to be related to my commit. (unless i'm misunderstanding the error) Any tips appreciated.

@ker2x Thanks for contributing :)
You are right - the error is unrelated to your change. It is a flaky test and I have linked the issue.
Restarting the workflows.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

Gradle Check (Jenkins) Run Completed with:

@kotwanikunal
Copy link
Member

#9776

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2023

Compatibility status:

Checks if related components are compatible with change 0662ffd

Incompatible components

Skipped components

Compatible components

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

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2023

Gradle Check (Jenkins) Run Completed with:

@ker2x
Copy link
Contributor Author

ker2x commented Sep 9, 2023

i have much more javadoc to commit, should i wait ?

@msfroh
Copy link
Collaborator

msfroh commented Sep 9, 2023

i have much more javadoc to commit, should i wait ?

Let's merge this one first.

Gradle checks are failing due to flaky tests. I've restarted the check run (see https://github.com/opensearch-project/OpenSearch/actions/runs/6113402537/job/16644851983?pr=9817), but I suspect that you may need to merge from main again to pick up the fixes for those tests. Sorry about that :(

Thank you so much for these fixes, @ker2x! These new Javadocs are so much better.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2023

Gradle Check (Jenkins) Run Completed with:

@ker2x
Copy link
Contributor Author

ker2x commented Sep 9, 2023

Thank you so much for these fixes, @ker2x! These new Javadocs are so much better.

Merged. i didn't make the mistake to commit on my main this time so i didn't mistakenly push more unrefined javadoc this time :) Let's see how it goes.

there are some javadoc with some "todo fix before commit" in this PR. I'll fix it in next PR (i shouldn't have disabled the "check for TODO" in inteliJ ^^)

thanks for the kind words. i'm having fun doing this. (i'm an Elastic database administrator switching to opensearch. i'm doing this for learning purpose)

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.smoketest.SmokeTestMultiNodeClientYamlTestSuiteIT.test {yaml=pit/10_basic/Delete all}

@codecov
Copy link

codecov bot commented Sep 10, 2023

Codecov Report

Merging #9817 (157dd0c) into main (c100c0c) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #9817      +/-   ##
============================================
- Coverage     71.16%   71.11%   -0.06%     
- Complexity    58114    58126      +12     
============================================
  Files          4824     4824              
  Lines        273897   273897              
  Branches      39910    39910              
============================================
- Hits         194926   194788     -138     
- Misses        62573    62800     +227     
+ Partials      16398    16309      -89     
Files Changed Coverage Δ
...main/java/org/opensearch/common/collect/Tuple.java 64.70% <ø> (ø)
...java/org/opensearch/common/crypto/DataKeyPair.java 0.00% <ø> (ø)
...h/common/crypto/DecryptedRangedStreamProvider.java 0.00% <ø> (ø)
...g/opensearch/compress/spi/CompressionProvider.java 100.00% <ø> (ø)
...cli/src/main/java/org/opensearch/cli/Terminal.java 74.73% <100.00%> (ø)

... and 469 files with indirect coverage changes

@msfroh msfroh merged commit 77f0df3 into opensearch-project:main Sep 10, 2023
@reta reta added the backport 2.x Backport to 2.x branch label Sep 10, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 10, 2023
* fixing javadoc of org.opensearch.cli.Terminal

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc of org.opensearch.cli.ExitCodes

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc

Signed-off-by: Laurent Laborde <[email protected]>

* fixing a mishap in formatting rule

Signed-off-by: Laurent Laborde <[email protected]>

---------

Signed-off-by: Laurent Laborde <[email protected]>
(cherry picked from commit 77f0df3)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
reta pushed a commit that referenced this pull request Sep 11, 2023
* fixing javadoc of org.opensearch.cli.Terminal



* fixing javadoc of org.opensearch.cli.ExitCodes



* fixing javadoc



* fixing a mishap in formatting rule



---------


(cherry picked from commit 77f0df3)

Signed-off-by: Laurent Laborde <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
kaushalmahi12 pushed a commit to kaushalmahi12/OpenSearch that referenced this pull request Sep 12, 2023
* fixing javadoc of org.opensearch.cli.Terminal

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc of org.opensearch.cli.ExitCodes

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc

Signed-off-by: Laurent Laborde <[email protected]>

* fixing a mishap in formatting rule

Signed-off-by: Laurent Laborde <[email protected]>

---------

Signed-off-by: Laurent Laborde <[email protected]>
Signed-off-by: Kaushal Kumar <[email protected]>
sarthakaggarwal97 pushed a commit to sarthakaggarwal97/OpenSearch that referenced this pull request Sep 20, 2023
* fixing javadoc of org.opensearch.cli.Terminal

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc of org.opensearch.cli.ExitCodes

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc

Signed-off-by: Laurent Laborde <[email protected]>

* fixing a mishap in formatting rule

Signed-off-by: Laurent Laborde <[email protected]>

---------

Signed-off-by: Laurent Laborde <[email protected]>
brusic pushed a commit to brusic/OpenSearch that referenced this pull request Sep 25, 2023
* fixing javadoc of org.opensearch.cli.Terminal

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc of org.opensearch.cli.ExitCodes

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc

Signed-off-by: Laurent Laborde <[email protected]>

* fixing a mishap in formatting rule

Signed-off-by: Laurent Laborde <[email protected]>

---------

Signed-off-by: Laurent Laborde <[email protected]>
Signed-off-by: Ivan Brusic <[email protected]>
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
* fixing javadoc of org.opensearch.cli.Terminal

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc of org.opensearch.cli.ExitCodes

Signed-off-by: Laurent Laborde <[email protected]>

* fixing javadoc

Signed-off-by: Laurent Laborde <[email protected]>

* fixing a mishap in formatting rule

Signed-off-by: Laurent Laborde <[email protected]>

---------

Signed-off-by: Laurent Laborde <[email protected]>
Signed-off-by: Shivansh Arora <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants