[GH-3193] Avoid deadlock in writeAllTo by removing WritableByteChannel usage

Author: wangyum

parquet-common/src/main/java/org/apache/parquet/bytes/ConcatenatingByteBufferCollector.java

@@ -23,8 +23,6 @@
 import java.io.IOException;
 import java.io.OutputStream;
 import java.nio.ByteBuffer;
-import java.nio.channels.Channels;
-import java.nio.channels.WritableByteChannel;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -72,9 +70,15 @@ public void close() {
 
   @Override
   public void writeAllTo(OutputStream out) throws IOException {
-    WritableByteChannel channel = Channels.newChannel(out);
     for (ByteBuffer buffer : slabs) {
-      channel.write(buffer.duplicate());
+      ByteBuffer dup = buffer.duplicate();
+      if (dup.hasArray()) {
+        out.write(dup.array(), dup.arrayOffset() + dup.position(), dup.remaining());
+      } else {
+        byte[] bytes = new byte[dup.remaining()];
+        dup.get(bytes);
+        out.write(bytes);
+      }
     }
   }