Skip to content

Conversation

@wstrzalka
Copy link

@wstrzalka wstrzalka commented Jun 10, 2025

Hi

This is proposal to fix the issue #80 raised by me some time ago.
It adds extra hook BlockReaderAdapter::failure(Exception ex) with option to either handle in the adapter or rethrow the exception.

@wstrzalka wstrzalka closed this Jun 24, 2025
@leonardehrenfried
Copy link
Collaborator

In #80 you talk about a broken input stream. Under which condition is the stream broken?

@wstrzalka
Copy link
Author

That may happen for example if you stream the pbf from the network (from some REST endpoint or S3).
The above patch is unfortunately broken though as InputStream.available() doesn't guarantees to return 0 only if there are no more bytes to read.

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.

2 participants