[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/jena/pull/157 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-19 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71346429 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/AbortableComparator.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software

Re: [GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-19 Thread A. Soroka
Sorry, that was meant to be on the line above the line with the variable `count`. See my edit. --- A. Soroka The University of Virginia Library > On Jul 19, 2016, at 10:25 AM, Chris Dollin > wrote: > >> Github user ajs6f commented on a diff in the pull request:

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-19 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71348900 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/AbortableComparator.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-19 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71347034 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/AbortableComparator.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-19 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71314599 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/SortedDataBag.java --- @@ -69,19 +69,81 @@ protected final ThresholdPolicy policy;

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-18 Thread afs
Github user afs commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71170935 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/SortedDataBag.java --- @@ -200,9 +263,13 @@ public void flush() if (!finishedAdding &&

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-18 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71170120 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/SortedDataBag.java --- @@ -69,19 +69,81 @@ protected final ThresholdPolicy policy;

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-18 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71167748 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/SortedDataBag.java --- @@ -69,19 +69,81 @@ protected final ThresholdPolicy policy;

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-18 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71167237 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/SortedDataBag.java --- @@ -69,19 +69,81 @@ protected final ThresholdPolicy policy;

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-18 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71161082 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/SortedDataBag.java --- @@ -69,19 +69,81 @@ protected final ThresholdPolicy policy;

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-18 Thread ajs6f
Github user ajs6f commented on a diff in the pull request: https://github.com/apache/jena/pull/157#discussion_r71160809 --- Diff: jena-arq/src/main/java/org/apache/jena/atlas/data/SortedDataBag.java --- @@ -69,19 +69,81 @@ protected final ThresholdPolicy policy;

[GitHub] jena pull request #157: Fixes for JENA-1212

2016-07-18 Thread ehedgehog
GitHub user ehedgehog opened a pull request: https://github.com/apache/jena/pull/157 Fixes for JENA-1212 Comparator used by sorting blocks of bindings now checks for cancellation every 1 compares and aborts the sort if cancelled=true. You can merge this pull request