You are viewing a plain text version of this content. The canonical link for it is here.

Posted to pr@jena.apache.org by GitBox <gi...@apache.org> on 2021/12/13 12:30:35 UTC

[GitHub] [jena] OyvindLGjesdal commented on a change in pull request #1130: xloader: set LC_ALL=C for external sort command

OyvindLGjesdal commented on a change in pull request #1130:
URL: https://github.com/apache/jena/pull/1130#discussion_r767714933



##########
File path: jena-db/jena-tdb2/src/main/java/org/apache/jena/tdb2/xloader/ProcIndexBuildX.java
##########
@@ -141,6 +141,7 @@ private static long sort_build_index(Logger LOG, String datafile, DatasetGraph d
             return 0;
         // Sort task.
         Process proc2;
+        ProcessBuilder pb2;

Review comment:
       Thanks for the comment @afs, moved inside the try block.




-- 
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.

To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org