neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#issuecomment-496660571
with regards to the "Commons Median" "the data must be at least partially
ordered." question. I did tests with a random set of 1,000,000,000
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#issuecomment-496651661
> If the query file is *.arq", it will be parsed are extended syntax.
>
> See jena-arq/testing/ARQ/GroupBy/count-01.arq, for example.
>
neumarcx commented on a change in pull request #568: Add Aggregate Median to
SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288260352
##
File path:
jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java
##
@@ -78,7 +82,7
ajs6f commented on a change in pull request #571: JENA-1714: Migrate 'migrate'
packages
URL: https://github.com/apache/jena/pull/571#discussion_r288198140
##
File path: jena-base/src/main/java/org/apache/jena/atlas/lib/StreamOps.java
##
@@ -34,25 +39,32 @@
return
ajs6f commented on a change in pull request #571: JENA-1714: Migrate 'migrate'
packages
URL: https://github.com/apache/jena/pull/571#discussion_r288197989
##
File path: jena-base/src/main/java/org/apache/jena/atlas/lib/StreamOps.java
##
@@ -34,25 +39,32 @@
return
ajs6f commented on a change in pull request #571: JENA-1714: Migrate 'migrate'
packages
URL: https://github.com/apache/jena/pull/571#discussion_r288197226
##
File path: jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpLib.java
##
@@ -18,31 +18,18 @@
package
afs commented on a change in pull request #571: JENA-1714: Migrate 'migrate'
packages
URL: https://github.com/apache/jena/pull/571#discussion_r288195909
##
File path: jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpLib.java
##
@@ -18,31 +18,18 @@
package
afs commented on a change in pull request #571: JENA-1714: Migrate 'migrate'
packages
URL: https://github.com/apache/jena/pull/571#discussion_r288195909
##
File path: jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpLib.java
##
@@ -18,31 +18,18 @@
package
ajs6f commented on a change in pull request #571: JENA-1714: Migrate 'migrate'
packages
URL: https://github.com/apache/jena/pull/571#discussion_r288177603
##
File path: jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpLib.java
##
@@ -18,31 +18,18 @@
package
ajs6f commented on a change in pull request #568: Add Aggregate Median to
SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288175406
##
File path:
jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java
##
@@ -78,7 +82,7 @@
afs commented on a change in pull request #568: Add Aggregate Median to SPARQL
ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288160427
##
File path:
jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java
##
@@ -78,7 +82,7 @@
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#issuecomment-496561733
If the query file is *.arq", it will be parsed are extended syntax.
See jena-arq/testing/ARQ/GroupBy/count-01.arq, for example.
For
On 27/05/2019 16:12, Marco Neumann wrote:
can you let us know what that entails on your end? is it primarily a matter
of managing branches on github?
Bandwidth! Better to do Europe-morning than later in the day.
We release from the master branch and, with the Jenkins jobs, should
always
neumarcx commented on a change in pull request #568: Add Aggregate Median to
SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288149932
##
File path:
jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java
##
@@ -0,0 +1,118
neumarcx commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#issuecomment-496540717
that's correct @afs thr syntax test is in place but I get an error (see at
the top of this pull request) for the GroupBy evaluation test. Is there
afs opened a new pull request #571: JENA-1714: Migrate 'migrate' packages
URL: https://github.com/apache/jena/pull/571
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
Andy Seaborne created JENA-1714:
---
Summary: Migrate'migrate' packages
Key: JENA-1714
URL: https://issues.apache.org/jira/browse/JENA-1714
Project: Apache Jena
Issue Type: Task
No, I missed something-- I didn't see that the evaluation test question was
unresolved, my mistake.
ajs6f
> On May 28, 2019, at 9:14 AM, Andy Seaborne wrote:
>
>
>
> On 28/05/2019 13:00, Marco Neumann wrote:
>> median function not going to be in this release?
>
> I see an outstanding
On 28/05/2019 13:00, Marco Neumann wrote:
median function not going to be in this release?
I see an outstanding request for evaluation tests on the PR. Did I miss
something?
This release is a direct follow-on to 3.11 when we got 3.11 out close to
schedule leaving GeoSPARQL to follow as
afs commented on issue #568: Add Aggregate Median to SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#issuecomment-496509838
I don't see any evaluation tests, only syntax tests.
Data can be provided by `VALUES ?x { 1 2 3 4 5 }` and `VALUES ?x { 1 2 3 4 5
6 }`.
afs commented on a change in pull request #568: Add Aggregate Median to SPARQL
ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288091834
##
File path: jena-arq/testing/ARQ/Syntax/syn-arq.sh
##
@@ -328,3 +328,17 @@ WHERE
GRAPH ?g { ?s ?p ?o.
OK sounds good, my next stats package will be released as a property
functions package I believe and not as part of ARQ. I will make changes to
the logging configuration tomorrow.
On Tue, May 28, 2019 at 1:33 PM ajs6f wrote:
> I'm afraid not. Honestly, I didn't realize that Andy was going to
ajs6f commented on a change in pull request #568: Add Aggregate Median to
SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288087992
##
File path:
jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java
##
@@ -0,0 +1,118 @@
neumarcx commented on a change in pull request #568: Add Aggregate Median to
SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288081119
##
File path:
jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java
##
@@ -0,0 +1,118
ajs6f commented on a change in pull request #568: Add Aggregate Median to
SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288079380
##
File path:
jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java
##
@@ -0,0 +1,118 @@
I'm afraid not. Honestly, I didn't realize that Andy was going to start
releasing so quickly. Time flies!
I've got a last request for you to stop using System.out.println to log, but
other than that it looks like you've responded to all the comments, so thank
you! We should be able to merge
ajs6f commented on a change in pull request #568: Add Aggregate Median to
SPARQL ARQ syntax
URL: https://github.com/apache/jena/pull/568#discussion_r288077331
##
File path:
jena-arq/src/main/java/org/apache/jena/sparql/expr/aggregate/AggMedian.java
##
@@ -0,0 +1,118 @@
median function not going to be in this release?
On Tue, May 28, 2019 at 10:25 AM Andy Seaborne wrote:
> +1
>
> > Please vote to approve this release:
> >
> > [ ] +1 Approve the release
> > [ ] 0 Don't care
> > [ ] -1 Don't release, because ...
>
--
---
Marco
+1
Please vote to approve this release:
[ ] +1 Approve the release
[ ] 0 Don't care
[ ] -1 Don't release, because ...
Hi,
Here is a vote on a release of Apache Jena 3.12.0.
This is the first proposed release candidate.
The deadline for the vote is Friday, 31th May, 2019 at 20:00 UTC
Release changes:
The main feature of this release is the new GeoSPARQL and Fuseki
GeoSPARQL modules.
The new NOTICE
30 matches
Mail list logo