You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@storm.apache.org by "SAY-5 (via GitHub)" <gi...@apache.org> on 2026/04/17 08:01:12 UTC

[PR] docs: fix 'occured' -> 'occurred' in LocallyCachedBlob Javadoc (storm)

SAY-5 opened a new pull request, #8530:
URL: https://github.com/apache/storm/pull/8530

   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.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@storm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] docs: fix 'occured' -> 'occurred' in LocallyCachedBlob Javadoc (storm)

Posted by "rzo1 (via GitHub)" <gi...@apache.org>.
rzo1 merged PR #8530:
URL: https://github.com/apache/storm/pull/8530


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@storm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] docs: fix 'occured' -> 'occurred' in LocallyCachedBlob Javadoc (storm)

Posted by "SAY-5 (via GitHub)" <gi...@apache.org>.
SAY-5 commented on PR #8530:
URL: https://github.com/apache/storm/pull/8530#issuecomment-4266458056

   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 `seperate` → `separate` in `worker-launcher/impl/configuration.h`.
   - `storm-client` — an `accross` → `across` in `trident/Stream.java`.
   - `flux` — a `recieve` → `receive` in `flux-wrappers/.../LogInfoBolt.java`.
   
   Three separate PRs, one per module, incoming in the next few minutes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@storm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] docs: fix 'occured' -> 'occurred' in LocallyCachedBlob Javadoc (storm)

Posted by "rzo1 (via GitHub)" <gi...@apache.org>.
rzo1 commented on PR #8530:
URL: https://github.com/apache/storm/pull/8530#issuecomment-4266377413

   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 🙃


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@storm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org