Skip to content

Fix PolarisEventListener javadoc#3607

Merged
dimas-b merged 1 commit intoapache:mainfrom
dimas-b:fix-javadoc
Jan 30, 2026
Merged

Fix PolarisEventListener javadoc#3607
dimas-b merged 1 commit intoapache:mainfrom
dimas-b:fix-javadoc

Conversation

@dimas-b
Copy link
Contributor

@dimas-b dimas-b commented Jan 28, 2026

  • Adjust javadoc to reference available methods.

Checklist

  • 🛡️ Don't disclose security issues! (contact security@apache.org)
  • 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
  • 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
  • 💡 Added comments for complex logic
  • 🧾 Updated CHANGELOG.md (if needed)
  • 📚 Updated documentation in site/content/in-dev/unreleased (if needed)

* Adjust javadoc to reference available methods.
@dimas-b
Copy link
Contributor Author

dimas-b commented Jan 28, 2026

CC: @adnanhemani , @olsoloviov

@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Jan 29, 2026
@olsoloviov
Copy link
Contributor

Thanks you, @dimas-b!

@dimas-b dimas-b merged commit 618d489 into apache:main Jan 30, 2026
15 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Jan 30, 2026
@dimas-b dimas-b deleted the fix-javadoc branch January 30, 2026 17:25
sungwy pushed a commit to sungwy/polaris that referenced this pull request Feb 7, 2026
* Adjust javadoc to reference available methods.
snazy added a commit to snazy/polaris that referenced this pull request Feb 11, 2026
* Fix PolarisEventListener javadoc (apache#3607)

* Adjust javadoc to reference available methods.

* Guides: add mandatory `curl` `--fail` option (apache#3610)

`curl` does _not_ return an error code when the URL can be retrieved but returns an error HTTP status code, for example 404, 403, 401. This means that the health checks are yieldíng "OK" too early.

This change adds the missing `--fail` option.

* Guides: compose dependencies / long-option (apache#3611)

Docker compose's "short" `depends_on` syntax (just mentioning the dependent service) does not assert on the expected condition (`service_healthy` or `service_completed_successfully`). This can lead to services being started too early, although their dependencies are not in the expected state.

This change updates the affected parts to use the "long" syntax.

* fix(deps): update dependency com.google.cloud:google-cloud-iamcredentials to v2.83.0 (apache#3635)

* chore(deps): update registry.access.redhat.com/ubi9/openjdk-21-runtime docker tag to v1.24-2.1769595317 (apache#3631)

* fix(deps): update dependency com.puppycrawl.tools:checkstyle to v13.1.0 (apache#3637)

* chore(deps): update amazon/aws-cli docker tag to v2.33.12 (apache#3638)

* fix(deps): update dependency com.azure:azure-sdk-bom to v1.3.4 (apache#3633)

* fix(deps): update dependency ch.qos.logback:logback-classic to v1.5.27 (apache#3632)

* chore(deps): update docker.io/openpolicyagent/opa docker tag to v1.13.1 (apache#3634)

* CI: Fix site workflow concurrency (apache#3642)

* Last merged commit 384be9e

---------

Co-authored-by: Dmitri Bourlatchkov <dmitri.bourlatchkov@gmail.com>
Co-authored-by: Mend Renovate <bot@renovateapp.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.

6 participants