You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jena.apache.org by xristy <gi...@git.apache.org> on 2017/11/28 18:37:11 UTC
[GitHub] jena pull request #318: JENA-1441 jena-text tracing for development
GitHub user xristy opened a pull request:
https://github.com/apache/jena/pull/318
JENA-1441 jena-text tracing for development
This adds tracing in `TextQueryPF` and `TextIndexLucene` to track calls to `exec` and subsequent processing as well as the add, update and delete of Lucene documents.
Also adds configuration item `text:cacheQueries` on `text:TextIndexLucene` which allows to bypass use of the `queryCache` in `TextQueryPF.query(...)` during development. This configuration item is not for general use and need not be documented further. `text:cacheQueries` defaults to `true` so there is no net change in the behavior of jena-text.
There is also tidying of code to use `doc.get()` rather than `doc.getValues() followed by vals[0]` in `TextLuceneIndex.query$`.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/BuddhistDigitalResourceCenter/jena highlighting
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/jena/pull/318.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #318
----
commit 7b016291a17cdcabb1cf7222094e1f00344ddf15
Author: Chris Tomlinson <ct...@moonvine.org>
Date: 2017-11-19T17:48:42Z
added config for using text cache; added debug log for Lucene docs
commit e2d6b89e369530a1c53d4579d500daa135439ab8
Author: Chris Tomlinson <ct...@moonvine.org>
Date: 2017-11-24T22:09:41Z
Merge remote-tracking branch 'upstream/master' into highlighting
commit e14457115e638b33dcd8923f22c2ff3655309ab0
Author: Chris Tomlinson <ct...@moonvine.org>
Date: 2017-11-25T00:22:58Z
use get vs getValues in query$; log trace for Lucene dev
commit 5e95a09eccaf7f8fa5ec659bebde83cdf8398163
Author: Chris Tomlinson <ct...@moonvine.org>
Date: 2017-11-26T16:19:11Z
Merge remote-tracking branch 'upstream/master' into highlighting
commit d98b096620ef09d115f745c9ecd1359d0a62c83e
Author: Chris Tomlinson <ct...@moonvine.org>
Date: 2017-11-28T17:52:50Z
Merge branch 'master' of https://github.com/apache/jena into highlighting
commit 452b3838aee02da4983e8df8c6b33c504e7c3f5a
Author: Chris Tomlinson <ct...@moonvine.org>
Date: 2017-11-28T17:57:16Z
added tracing exec, etc for development
----
---
[GitHub] jena issue #318: JENA-1441 jena-text tracing for development
Posted by xristy <gi...@git.apache.org>.
Github user xristy commented on the issue:
https://github.com/apache/jena/pull/318
Closing to reissue without the JENA-1438 commit
---
[GitHub] jena issue #318: JENA-1441 jena-text tracing for development
Posted by afs <gi...@git.apache.org>.
Github user afs commented on the issue:
https://github.com/apache/jena/pull/318
FYI: On a PR branch, you can do all sorts of git stuff on your local repo like reset to remove a commit, rebasing, and squashing. Then do a force push to your PR branch at GH and it will all appear here with the updated state. GH even tracks revisions and hides comments no longer relevant. (Use all this with care!)
---
[GitHub] jena pull request #318: JENA-1441 jena-text tracing for development
Posted by xristy <gi...@git.apache.org>.
Github user xristy closed the pull request at:
https://github.com/apache/jena/pull/318
---