You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/03/15 02:44:25 UTC
[GitHub] [pulsar] hangc0276 opened a new pull request #14685: Fix incorrect entryId in warn log when reading entry from tiered storage
hangc0276 opened a new pull request #14685:
URL: https://github.com/apache/pulsar/pull/14685
### Motivation
When read offload data failed, it prints a warn log. However, the actual read entryId doesn't print correctly, which will makes hard to debug.
```
WARN org.apache.bookkeeper.mledger.offload.jcloud.impl.BlobStoreBackedReadHandleImpl - Read an unexpected entry id [] which is smaller than the next expected entry id 49648, seeking to the right position
```
### Modification
1. Print correct actual entryId
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] github-actions[bot] commented on pull request #14685: Fix incorrect entryId in warn log when reading entry from tiered storage
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14685:
URL: https://github.com/apache/pulsar/pull/14685#issuecomment-1067542968
@hangc0276:Thanks for providing doc info!
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Shoothzj merged pull request #14685: Fix incorrect entryId in warn log when reading entry from tiered storage
Posted by GitBox <gi...@apache.org>.
Shoothzj merged pull request #14685:
URL: https://github.com/apache/pulsar/pull/14685
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] github-actions[bot] commented on pull request #14685: Fix incorrect entryId in warn log when reading entry from tiered storage
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14685:
URL: https://github.com/apache/pulsar/pull/14685#issuecomment-1067509623
@hangc0276:Thanks for your contribution. For this PR, do we need to update docs?
(The [PR template contains info about doc](https://github.com/apache/pulsar/blob/master/.github/PULL_REQUEST_TEMPLATE.md#documentation), which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] hangc0276 commented on pull request #14685: Fix incorrect entryId in warn log when reading entry from tiered storage
Posted by GitBox <gi...@apache.org>.
hangc0276 commented on pull request #14685:
URL: https://github.com/apache/pulsar/pull/14685#issuecomment-1067530324
> @hangc0276 seems that we do not update docs for this PR？
@Anonymitaet Yes, it doesn't need doc updates.
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Anonymitaet commented on pull request #14685: Fix incorrect entryId in warn log when reading entry from tiered storage
Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on pull request #14685:
URL: https://github.com/apache/pulsar/pull/14685#issuecomment-1067518389
@hangc0276 seems that we do not update docs for this PR？
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org