Skip to content

Restore the third callout entry in the Hibernate ORM interceptor example - #56514

Open
rolfedh wants to merge 1 commit into
quarkusio:mainfrom
rolfedh:docs-callouts-hibernate-orm
Open

Restore the third callout entry in the Hibernate ORM interceptor example#56514
rolfedh wants to merge 1 commit into
quarkusio:mainfrom
rolfedh:docs-callouts-hibernate-orm

Conversation

@rolfedh

@rolfedh rolfedh commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Hibernate ORM guide, Interceptors example: the <3> marker on the onLoad method had no callout entry, so the rendered page shows a circled 3 that points nowhere, and a strict AsciiDoc parser rejects the page.

The 2021 original had <2> Either extend org.hibernate.EmptyInterceptor or implement org.hibernate.Interceptor directly. and <3> Implement methods as necessary.. #47184 replaced EmptyInterceptor with Interceptor and folded both entries into one <2>, leaving the <3> marker behind. This restores the split: <2> says to implement org.hibernate.Interceptor, whose methods all have default implementations, and <3> names onLoad as the example callback.

Removing the <3> marker instead is fine by me if you prefer.

Part of #56509

The <3> marker on the onLoad method lost its entry when quarkusio#47184 folded
the EmptyInterceptor entry and the "implement methods" entry into one.
Split them again so every marker has an entry directly under the block.

Part of quarkusio#56509
@quarkus-bot

quarkus-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 8f0b3b5.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

🎊 PR Preview 382dc35 has been successfully built and deployed to https://quarkus-pr-main-56514-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

@rolfedh
rolfedh requested a review from FroMage September 9, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants