Exception stack trace overwritten

[raduioanstoica]

Catch all clauses sometimes obfuscate the original exception. For example, in client/src/main/java/com/ibm/crail/CrailBufferedInputStream.java:230, the catch clause creates and throws a new exception which results in loosing the actual stack trace:

       private void triggerFetch() throws IOException {
                try {
                        if (future == null && internalBuf.remaining() == 0){
                                internalBuf.clear();
                                future = inputStream.read(internalBuf);
                                if (future == null){
                                        internalBuf.clear().flip();
                                }
                        }
                } catch(Exception e) {
                        throw new IOException(e);
                }
        }

A simple solution would be to print the original stack trace. A better solution would be to remove the try-catch and force the enclosed code to properly deal with all non-IOExceptions.

[patrickstuedi]

Inner exception can always be accessed, stacktrace is maintained.