Skip to content

docs: fix 'occured' -> 'occurred' in LocallyCachedBlob Javadoc#8530

Merged
rzo1 merged 1 commit intoapache:masterfrom
SAY-5:fix-occured-occurred-typo2
Apr 17, 2026
Merged

docs: fix 'occured' -> 'occurred' in LocallyCachedBlob Javadoc#8530
rzo1 merged 1 commit intoapache:masterfrom
SAY-5:fix-occured-occurred-typo2

Conversation

@SAY-5
Copy link
Copy Markdown
Contributor

@SAY-5 SAY-5 commented Apr 17, 2026

Trivial spelling fix on the cleanup() Javadoc in storm-server/.../localizer/LocallyCachedBlob.java:

This will be called after updating a blob, either successfully or if an error has occurred.

No functional changes.

Single-word spelling correction on the cleanup() method Javadoc.

No functional changes.

Signed-off-by: Sai Asish Y <say.apm35@gmail.com>
@rzo1
Copy link
Copy Markdown
Contributor

rzo1 commented Apr 17, 2026

Thx for the PR. I would appreciate if you could provide typo fixes on a per module basis instead of doing a commit/pr per identified issue. This will reduce workload on our side 🙃

@SAY-5
Copy link
Copy Markdown
Contributor Author

SAY-5 commented Apr 17, 2026

Thanks @rzo1, got it — per-module batching makes sense. This PR is already scoped to storm-server (one Javadoc in LocallyCachedBlob is the only typo I found in that module). I'll open separate per-module PRs for the remaining typos I spotted:

  • storm-core — a seperateseparate in worker-launcher/impl/configuration.h.
  • storm-client — an accrossacross in trident/Stream.java.
  • flux — a recievereceive in flux-wrappers/.../LogInfoBolt.java.

Three separate PRs, one per module, incoming in the next few minutes.

@rzo1 rzo1 merged commit 7423651 into apache:master Apr 17, 2026
@reiabreu reiabreu added this to the 2.8.7 milestone Apr 19, 2026
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.

3 participants