[SPARK-21181] Release byteBuffers to suppress netty error messages
authorDhruve Ashar <dhruveashar@gmail.com>
Fri, 23 Jun 2017 17:36:29 +0000 (10:36 -0700)
committerMarcelo Vanzin <vanzin@cloudera.com>
Fri, 23 Jun 2017 17:36:29 +0000 (10:36 -0700)
commit1ebe7ffe072bcac03360e65e959a6cd36530a9c4
tree7ceb1c6bfb3a7798244564e55419acbf636ff6dc
parentb803b66a8133f705463039325ee71ee6827ce1a7
[SPARK-21181] Release byteBuffers to suppress netty error messages

## What changes were proposed in this pull request?
We are explicitly calling release on the byteBuf's used to encode the string to Base64 to suppress the memory leak error message reported by netty. This is to make it less confusing for the user.

### Changes proposed in this fix
By explicitly invoking release on the byteBuf's we are decrement the internal reference counts for the wrappedByteBuf's. Now, when the GC kicks in, these would be reclaimed as before, just that netty wouldn't report any memory leak error messages as the internal ref. counts are now 0.

## How was this patch tested?
Ran a few spark-applications and examined the logs. The error message no longer appears.

Original PR was opened against branch-2.1 => https://github.com/apache/spark/pull/18392

Author: Dhruve Ashar <dhruveashar@gmail.com>

Closes #18407 from dhruve/master.
common/network-common/src/main/java/org/apache/spark/network/sasl/SparkSaslServer.java