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
PR for 3.5.2: https://github.com/caskdata/cdap/pull/6856
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:
