New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added byteLength to Metadata #599

Merged
merged 1 commit into from Aug 15, 2018

Conversation

Projects
None yet
2 participants
@macheteaustria
Contributor

macheteaustria commented Aug 7, 2018

FetcherBolt already read the byteLength from the response, but didn't add it to Metadata.
(This could be helpful if you want to index the document size)

@jnioche

This comment has been minimized.

Member

jnioche commented Aug 9, 2018

Thanks, could you please add the same to SimpleFetcherBolt as well? Bear in mind that this is the length fetched which may have been trimmed, not necessarily the real size of the document.

@jnioche jnioche added this to the 1.11 milestone Aug 9, 2018

@jnioche jnioche merged commit 79e2f2a into DigitalPebble:master Aug 15, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jnioche

This comment has been minimized.

Member

jnioche commented Aug 15, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment