Replace usages of ChunkResponder with BodyProducer in LogHandler

Description

Implement the changes of CDAP-5080, for LogHandler.

Release Notes

The Log HTTP Handler and Router have been fixed to allow the streaming of larger logs files.

Activity

Show:
Ali Anwar
September 30, 2016, 3:42 AM
Edited
Ali Anwar
October 5, 2016, 2:36 AM

release/3.5 was merged into release/3.6, so the fix is there also:
https://github.com/caskdata/cdap/pull/6884

Poorna Chandra
October 11, 2016, 11:46 PM

Re-opening this issue to test the the behavior of the new implementation when an exception is thrown in the body producer nextChunk. The expected behavior is for the chunked connection to be closed cleanly by sending an empty chunk.

Ali Anwar
October 17, 2016, 7:14 PM

Verified that if I throw an exception from the nextChunk method, that the curl command making the http request doesn't hang; instead, I get the following as output:

Fixed

Assignee

Ali Anwar

Reporter

Ali Anwar

Labels

None

Docs Impact

None

UX Impact

None

Components

Fix versions

Affects versions

Priority

Major
Configure