Github user afs commented on the issue:
https://github.com/apache/jena/pull/345
Use new operation `TableEmpty.isTableEmpty` to test whether a data
structure is the empty table, not rely on the contents. Thus a table of no rows
but with declared variables, keeps the variables in the
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/346
JENA-1469: Adjust internal tracking state across promotion
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena txn-promote2
Alternatively you
Github user afs commented on the issue:
https://github.com/apache/jena/pull/345
It does not need to change. It will behave as before.
The changes here differentiate between a table that is of class
`TableEmpty`, using function `TableEmpty.isTableEmpty`, and a table that
Github user afs commented on the issue:
https://github.com/apache/jena/pull/345
@RickMoynihan : thanks for looking at it.
Merge : soon, because you provide independent checking. It will then be in
the daily development build but that is not a formal release.
Release
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/347
No instances for TDB and TDB2 classes
These classes are content and function holders only.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/348
JENA-1471: Fix to release old tasks records.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena backup-21
Alternatively you can review and
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/349
Cleanup comments and code around txn
* Comment that is out of date
* Code that has legacy use only
You can merge this pull request into a Git repository by running:
$ git pull https
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/349#discussion_r165013487
--- Diff:
jena-tdb/src/main/java/org/apache/jena/tdb/store/GraphNonTxnTDB.java ---
@@ -32,25 +34,58 @@
* @see GraphTxnTDB
*/
public class
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/350
JENA-1475: Build with Java9
Top level POM changes to be able to build with Java9, output for Java8.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user afs commented on the issue:
https://github.com/apache/jena/pull/350
This PR is currently for discussion and is not ready to be merged because
it switches out Elephas.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/351
JENA-1476: Switch javax.xml.bind.DatatypeConverter to commons-codec
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena remove-jaxb
Github user afs commented on the issue:
https://github.com/apache/jena/pull/351
The use by Jena is quite simple, string encoding for base64 and hex binary,
and is covered by commons-codec.
This PR adds commons-codec as a dependency for jena-core - it was already a
managed
Github user afs commented on the issue:
https://github.com/apache/jena/pull/350
Elephas is enabled.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/354
JENA-1482: Binding checks
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena binding-check
Alternatively you can review and apply these
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/355
JENA-1483: Promote with explicit promotion mode.
Includes a fix for JENA-1478.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena promote-arg
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/355#discussion_r167362604
--- Diff: pom.xml ---
@@ -735,6 +735,7 @@
maven-compiler-plugin
3.7.0
+false
--- End diff
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/356
JENA-1474: Update fApache parent POM v18 -> v19
Issues:
* jena-maven-tools does not pass its tests. This module is commented out.
(This has happened before on an Apeche parent update.)
*
Github user afs closed the pull request at:
https://github.com/apache/jena/pull/354
---
Github user afs commented on the issue:
https://github.com/apache/jena/pull/354
This has got POM early work on changes for Apache-v19 mixed into it. i
will close this and submit a clean PR.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/357
JENA-1482: Binding checks
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena binding-check2
Alternatively you can review and apply these
Github user afs commented on the issue:
https://github.com/apache/jena/pull/357
Sorry about that - closing and redoing the PR lost the split.
From the previous PR#354, see commit for 6abd485868376 for the null checks
and 8f17867269f5e for the reformatting.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/359
Fixes for transactions after JENA-1458
Always check autogenerated code!
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena ds-impl-fixes
Github user afs commented on the issue:
https://github.com/apache/jena/pull/359
Not in the 3.6.0 release.
---
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/360#discussion_r167613683
--- Diff: jena-text/src/main/java/jena/textindexer.java ---
@@ -104,39 +105,57 @@ protected String getSummary() {
@Override
protected void
Github user afs commented on the issue:
https://github.com/apache/jena/pull/360
This is in the code base now.
Can we close it please?
To do it after the event, you can do an empty commit with message "This
closes #360."
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/361
Small improvements
1. Yet another check for JENA-1482
2. Put some cut-and-paste internal code as compiled java so it stay up to
date.
The (2) code is because Jena does not do multiple
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/361#discussion_r168597752
--- Diff: jena-tdb/src/main/java/org/apache/jena/tdb/solver/BindingTDB.java
---
@@ -120,6 +121,8 @@ public Node get1(Var var)
if ( id == null
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/364
JENA-1482: tests for null NodeIds at the TDB level
Showing up in the build in jena-text as
`org.apache.jena.tdb.TDBException: No node in NodeTable for NodeId
[DoesNotExist]`
You can merge this
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/365
JENA-1490: Handle blank nodes client-server when using Fuseki.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena fuseki-connect
Alternatively
Github user afs commented on the issue:
https://github.com/apache/jena/pull/365
Note: WIP at this time.
There are tests to be tidied up and put into the PR.
---
Github user afs commented on the issue:
https://github.com/apache/jena/pull/365
Final changes now made.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/366
JENA-1494: Reader properties
Also includes:
* Additional operations for RDFParser and RDFWriter
* RDF/XML Writer properties example.
You can merge this pull request into a Git repository by
Github user afs commented on the issue:
https://github.com/apache/jena/pull/365
Add controllable parse-check of SPARQL request strings.
---
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/366#discussion_r170642885
--- Diff:
jena-arq/src/main/java/org/apache/jena/riot/lang/ReaderRIOTRDFXML.java ---
@@ -116,17 +154,29 @@ public void parse() {
arp.getHandlers
Github user afs commented on the issue:
https://github.com/apache/jena/pull/363
Please resolve the JIRA. Thanks.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/369
JENA-1492: Pass down transactions from general datasets
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena txn-general
Alternatively you can
Github user afs commented on the issue:
https://github.com/apache/jena/pull/369
"Skolemization function" is not related; it is a small clarification done
in passing.
---
Github user afs commented on the issue:
https://github.com/apache/jena/pull/369
Yes, this is models passing down transactions. It only became possible
with dataset-transaction promotion (new to 3.7.0).
This all has a lot of knock on consequences. The wrap/create
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/369#discussion_r172271582
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/core/TxnDataset2Graph.java ---
@@ -0,0 +1,240 @@
+/*
+ * Licensed to the Apache Software
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/369#discussion_r172275879
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/core/TxnDataset2Graph.java ---
@@ -0,0 +1,240 @@
+/*
+ * Licensed to the Apache Software
Github user afs commented on the issue:
https://github.com/apache/jena/pull/371
Git has decided this is a rename when it is actually a delete/new file.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/371
JENA-1498: Fuseki backup script
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena fuseki-backup-script
Alternatively you can review and apply
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/372
Reformatting + two RiotChars functions.
Minor : reformatting around blank node allocation and two library
functions, triggered by checking blank nodes formats.
* `RiotChars.isHexCharLC
Github user afs commented on the issue:
https://github.com/apache/jena/pull/353
Thank you to @santiago-a-vidal for this work! It is really interesting to
have Bandago prototype applied the code base. Given the age of Jena, and
evolution of contributors skills, I'm sure that
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/372#discussion_r172610615
--- Diff:
jena-arq/src/main/java/org/apache/jena/riot/lang/BlankNodeAllocatorGlobal.java
---
@@ -18,41 +18,39 @@
package org.apache.jena.riot.lang
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/373
JENA-1454: Fix to handle text output format.
Bug fix to JENA-1454.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena fuseki-text
Alternatively
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/373#discussion_r172680400
--- Diff:
jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/servlets/ResponseResultSet.java
---
@@ -131,26 +130,30 @@ public static void
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
The JSON ecosystem moves fast! GraphQL is now significant and how that
might be used is also very interesting. There are different ways to use GraphQL
and maybe it is not the solution to all needs anyway
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/376
JENA-1500: Plain in-memory dataset and empty graphs.
This PR will cause tests to fail under #374 goes in.
You can merge this pull request into a Git repository by running:
$ git pull https
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/377#discussion_r173497971
--- Diff:
jena-core/src/test/java/org/apache/jena/graph/test/TestTypedLiterals.java ---
@@ -989,7 +991,17 @@ public void testXSDanyURI() {
Node node2
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/378
JENA-1504: Fix batch handling.
Fix batch handling.
Fix new line output.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena trig-blocks
Github user afs commented on the issue:
https://github.com/apache/jena/pull/368
Closing this PR. I see these commits in the codebase under different SHA1
ids (b8b482ecaa and 036ebce8ed).
@jaco0646 - thanks!
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/379
Refactor tests; don't test for GregorianYear< 0
Follow on from JENA-1503.
This splits up some `TestTypeLiteral` tests and removes the old testing for
GregorianYear< 0 which didn'
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/380
JENA-1480: Use $.getJSON(url,data)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena jena-1480-strict-urls
Alternatively you can review and
Github user afs commented on the issue:
https://github.com/apache/jena/pull/379
Added zero. I split one mega test (`testDateTime`) into parts. It was a
sequence of independent parts. The same testing is done so that it is easier to
work on any failures. `testDateTime_9` contains a
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
I applied 114.patch locally and it integrated just fine.
I tried `JSON { "s": ?s , "p": ?p , "o" : ?o } WHERE { ?s ?p ?o }` on data
```
:s :p 123 .
:s
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
Tests?
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/381
Reformat QueryIterGroup; no test out from TestParameterizedSparqlString
Reformat `QueryIterGroup` prior to work on aggregates.
Remove print out in from `TestParameterizedQueryString`.
Remove a
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/382
JENA-1507: Fixes for aggregates when no WHERE match.
This includes the two commits of PR #381.
Commit 8a64ffee are the actual functional changes for JENA-1507.
You can merge this pull request
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
Conversion code (draft): 2dd4418a22 (on a branch in my GH at the moment).
The code is in `RDFTerm2Json.fromNode`.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/383
JENA-632: Conversion of Nodes to JsonValues.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena node2json
Alternatively you can review and
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
PR #383 - we can still change it based on experience.
BTW The grammar accepts the JSON field names in single quotes to make
embedding in strings easier. It's a STRING1, either one `&quo
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/384
JENA-1158: Make the union graph a graphView(Quad.unionGraph).
Don't rely on the default implementation in DatasetGraphBase.
You can merge this pull request into a Git repository by running:
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
Great timing! but as ever, it it makes it, it makes it, and if it doesn't,
it doesn't.
(I will not be able to build 3.7.0 until next week anyway.)
PR #383 merged.
---
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/384#discussion_r174896205
--- Diff: jena-arq/src/main/java/org/apache/jena/sparql/core/GraphView.java
---
@@ -146,7 +148,7 @@ private static Node graphNode(Node gn) {
public void
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/387#discussion_r175388799
--- Diff:
jena-rdfconnection/src/main/java/org/apache/jena/rdfconnection/RDFConnectionRemote.java
---
@@ -280,7 +280,7 @@ private void updateExec(UpdateRequest
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175409690
--- Diff:
jena-arq/src/main/java/org/apache/jena/query/ResultSetFormatter.java ---
@@ -516,6 +520,41 @@ static public void outputAsJSON(boolean booleanResult
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175910160
--- Diff: jena-arq/Grammar/master.jj ---
@@ -2176,6 +2251,19 @@ String String() : { Token t ; String lex ; }
}
}
+#ifdef ARQ
+Number
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175911878
--- Diff: jena-arq/src/main/java/org/apache/jena/query/Query.java ---
@@ -515,6 +519,14 @@ public void addHavingCondition(Expr expr)
havingExprs.add
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175906815
--- Diff: jena-arq/Grammar/master.jj ---
@@ -99,7 +99,42 @@ import org.apache.jena.sparql.core.Quad ;
#endif
public class CLASS extends PARSERBASE
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175910819
--- Diff: jena-arq/Grammar/master.jj ---
@@ -99,7 +99,42 @@ import org.apache.jena.sparql.core.Quad ;
#endif
public class CLASS extends PARSERBASE
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175915889
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/syntax/syntaxtransform/QueryTransformOps.java
---
@@ -217,6 +217,11 @@ public void visitAskResultForm
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175913506
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/engine/http/QueryEngineHTTP.java
---
@@ -532,6 +535,18 @@ else if (
actualContentType.equals
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175914165
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/serializer/QuerySerializer.java
---
@@ -142,7 +144,19 @@ public void visitAskResultForm(Query query
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175912825
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/engine/JsonIterator.java ---
@@ -0,0 +1,100 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175912074
--- Diff: jena-arq/src/main/java/org/apache/jena/query/Query.java ---
@@ -515,6 +519,14 @@ public void addHavingCondition(Expr expr)
havingExprs.add
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175915508
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/serializer/QuerySerializer.java
---
@@ -142,7 +144,19 @@ public void visitAskResultForm(Query query
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r175909873
--- Diff: jena-arq/Grammar/master.jj ---
@@ -326,6 +361,46 @@ void AskQuery() : {}
SolutionModifier()
}
+#ifdef ARQ
+void JsonQuery
Github user afs commented on the issue:
https://github.com/apache/jena/pull/385
Workflow doucmentation:
https://cwiki.apache.org/confluence/display/JENA/Commit+Workflow+for+Github-ASF
---
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
Syntax tests: https://gist.github.com/afs/e069478ec6839267256b9eacd30aa2c4
---
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
Eval tests: https://gist.github.com/afs/9e57b705a0d53889606346f6723238ce
Framework : needs more tests!
---
Github user afs commented on the issue:
https://github.com/apache/jena/pull/114
If it does not make the release then - whatever! - there'll be another
along soon!
I may be able to do some work on it after the release. I think I now
understand it better and it makes a u
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/114#discussion_r176029548
--- Diff: jena-arq/Grammar/master.jj ---
@@ -2176,6 +2251,19 @@ String String() : { Token t ; String lex ; }
}
}
+#ifdef ARQ
+Number
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/388
Some more RDFConnection examples
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena rdfconnection-examples
Alternatively you can review and
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/389
JENA-1513: Call initialization on ResultSet related paths
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena resultset-init
Alternatively you
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/391
Jena 1516 object file storage
The main commit is the second one.
Commit 1 is general removal of the alloc-complete writer cycle.
Commit 2 is the core fix - simpler wring of objects
Github user afs commented on the issue:
https://github.com/apache/jena/pull/390
Hi there - thanks for the PR. There is a problem somewhere.
Looking at the RDF tutorial, it uses "http\://somewhere/JohnSmith" (no /)
so I think the problem in the data file vc-db-
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/391#discussion_r179411949
--- Diff: jena-tdb/src/main/java/org/apache/jena/tdb/lib/NodeLib.java ---
@@ -53,26 +52,32 @@
// Characters in IRIs that are illegal and cause SSE
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/392
Add comments to NodeLib.workspace (from PR#391), reformat, remove dead code.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena
Github user afs commented on the issue:
https://github.com/apache/jena/pull/392
In response to review of #391.
---
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/396
JENA-1520: tdb2.tdbstats: cmd and fix for rdf:type
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena tdb2-tdbstats
Alternatively you can
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/397
TDB2 tidyup
Including JENA-1521 (TDB1 and TDB2 close).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena tdb2-tidyup
Alternatively you can
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/394#discussion_r181393545
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpWalker.java ---
@@ -139,6 +139,7 @@ protected void visitN(OpN op) {
@Override
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/398
JENA-1523: Allow internal variables names in variable parsing.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena var-databag
Alternatively
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/398#discussion_r181455972
--- Diff:
jena-arq/src/main/java/org/apache/jena/riot/tokens/TokenizerText.java ---
@@ -739,16 +739,21 @@ private String readWordSub(boolean
leadingDigitAllowed
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/396#discussion_r181468935
--- Diff:
jena-db/jena-tdb2/src/main/java/org/apache/jena/tdb2/solver/stats/StatsCollectorNodeId.java
---
@@ -18,39 +18,39 @@
package
Github user afs commented on the issue:
https://github.com/apache/jena/pull/399
Hi there - could you please split out the automatic module name name part
then we can get that into the codebase soon and separately from the API impact
of the rest?
I've create JENA-1525 fo
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/400#discussion_r181818306
--- Diff: jena-db/pom.xml ---
@@ -75,6 +79,13 @@
org.apache.maven.plugins
maven-jar-plugin
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/401#discussion_r182099487
--- Diff:
apache-jena-osgi/jena-osgi/src/main/resources/META-INF/services/org.apache.jena.system.JenaSubsystemLifecycle
---
@@ -1,4 +1,4
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/401#discussion_r182097398
--- Diff: jena-arq/src/main/java/org/apache/jena/system/JenaInit.java ---
@@ -0,0 +1,40 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under
901 - 1000 of 1218 matches
Mail list logo