cgivre commented on a change in pull request #1778: Drill-7233: Format Plugin 
for HDF5
URL: https://github.com/apache/drill/pull/1778#discussion_r363949309
 
 

 ##########
 File path: 
contrib/format-hdf5/src/main/java/org/apache/drill/exec/store/hdf5/writers/HDF5MapDataWriter.java
 ##########
 @@ -52,8 +52,8 @@ public HDF5MapDataWriter(IHDF5Reader reader, RowSetLoader 
columnWriter, String d
     } catch (Exception e) {
       throw UserException
         .dataReadError()
-        .message("Error writing Compound Field: ")
-        .addContext("Error", e.getMessage())
+        .message("Error writing Compound Field: {}", e.getMessage())
+        .addContext("Error: {}", e.getStackTrace())
 
 Review comment:
   Fixed that and found a few more string formats in UserExceptions. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to