Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QueryUtils circumvents query processor caching for non-tolog languages #391

Closed
GoogleCodeExporter opened this issue Mar 16, 2015 · 5 comments

Comments

@GoogleCodeExporter
Copy link

The only way to get a non-tolog query processor is:

QueryUtils.getQueryProcessorFactory("TMQL").createQueryProcessor(...);

This creates a new processor every time it's called and thus circumvents the 
caching functionality implemented in QueryUtils.

Original issue reported on code.google.com by pbrandw...@gmail.com on 4 May 2011 at 10:24

@GoogleCodeExporter
Copy link
Author

Original comment by qsieb...@gmail.com on 4 May 2011 at 10:26

  • Added labels: Priority-Low, Component-Engine, Newbie
  • Removed labels: Priority-Medium

@GoogleCodeExporter
Copy link
Author

Original comment by qsieb...@gmail.com on 4 May 2011 at 3:17

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

We have created a patch for this issue. New methods were added to QueryUtils to 
request a query processor for a specific language. In order for the caching of 
query processors to be able to distinct different language processors of a 
topic map, we have added an extra mapping in qpcache.

These additions to QueryUtils make it specifiy what query language you want to 
use. This code is backwards compatible with the previous version of QueryUtils. 
So no code that uses QueryUtils will have to be changed.

Original comment by pbrandw...@gmail.com on 19 May 2011 at 8:22

Attachments:

@GoogleCodeExporter
Copy link
Author

Fixed in r2113

Original comment by qsieb...@gmail.com on 19 May 2011 at 9:47

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Original comment by qsieb...@gmail.com on 27 Jan 2012 at 11:03

  • Added labels: Release5.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant