Thanks Mark, patch looks good. +1
Two minor problems:
- There is an additional System.out.println("length of response: " + e.length) in OpenEntity.
- In ByteRangeInputStream, the update(..) header splits into two lines. It should be
+ private int update(final int n) throws IOException {
- throws IOException {
instead of
+ private int update(final int n)
throws IOException {
Since both are very minor and I have changed them in my machine, let me post a patch.
The patch (HDFS-4549.2.patch) looks different from the current trunk code. For backporting, we would try to backport the code exactly so that it is easier to maintain the code. Could you make the backport patch the same as the original patches? Some other comments:
+ return update(getInputStream().read())