[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-11-05 Thread Michael McCandless (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14197983#comment-14197983
 ] 

Michael McCandless commented on LUCENE-5889:


I think we shouldn't backport commit to 4.10.x?  That branch is only for bug 
fixes... hopefully we soon release 5.0, which has the new commit method.

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, Trunk

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-11-04 Thread Mike Sokolov (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14197460#comment-14197460
 ] 

Mike Sokolov commented on LUCENE-5889:
--

It seems like my issues got addressed here - thanks! It looks like possibly 
this is ready for backporting to 4.x now?  I would love to see this in a 
release if anyone has a moment to merge it in.

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, Trunk

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-24 Thread Shawn Heisey (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14108446#comment-14108446
 ] 

Shawn Heisey commented on LUCENE-5889:
--

There's a precommit failure for missing javadoc on 
AnalyzingInfixSuggester#commit(), added by the svn commit for this issue.  If I 
add a dummy javadoc to the method, precommit passes.  I would fix, but I don't 
really know what the javadoc should say, and I'd rather not add something 
that's incorrect or incomplete.


 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-24 Thread Michael McCandless (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14108512#comment-14108512
 ] 

Michael McCandless commented on LUCENE-5889:


Grr, sorry for all the noise here :(

I'll fix the javadocs smoke test failure.

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-24 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14108518#comment-14108518
 ] 

ASF subversion and git services commented on LUCENE-5889:
-

Commit 1620180 from [~mikemccand] in branch 'dev/trunk'
[ https://svn.apache.org/r1620180 ]

LUCENE-5889: add missing javadocs

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-24 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14108524#comment-14108524
 ] 

ASF subversion and git services commented on LUCENE-5889:
-

Commit 1620182 from [~mikemccand] in branch 'dev/branches/branch_4x'
[ https://svn.apache.org/r1620182 ]

LUCENE-5889: add missing javadocs

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-22 Thread Mark Miller (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14107166#comment-14107166
 ] 

Mark Miller commented on LUCENE-5889:
-

Is there a thread safety issue with the writer in AnalyzingInfixSuggester here?

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-22 Thread Robert Muir (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14107201#comment-14107201
 ] 

Robert Muir commented on LUCENE-5889:
-

FYI Mike just left on vacation for a while. I'm not familiar with the issue yet 
so unable to help right now myself (currently busy).

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-22 Thread Areek Zillur (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14107212#comment-14107212
 ] 

Areek Zillur commented on LUCENE-5889:
--

I am currently investigating the problem, will update when I find anything 
interesting.

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-22 Thread Hoss Man (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14107220#comment-14107220
 ] 

Hoss Man commented on LUCENE-5889:
--

bq. Is there a thread safety issue with the writer in AnalyzingInfixSuggester 
here?

there's definitely a lock not getting freed somewhere - but i don't know about 
thread safety.

bq. I am currently investigating the problem, will update when I find anything 
interesting.

Thanks areek.

NOTE: it's totally possible that this change itself is fine and correct, and 
just happened to tickle some existing bug in the Solr factories that wrap the 
suggesters


 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-22 Thread Mark Miller (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14107228#comment-14107228
 ] 

Mark Miller commented on LUCENE-5889:
-

bq. but i don't know about thread safety.

It's hard to know at a glance, but the class is written in a way that says it 
expects to be used by multiple threads and the indexwriter is not accessed in a 
way that expects to be used by multiple threads. No fire, but a bunch of smoke.

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-22 Thread Areek Zillur (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14107391#comment-14107391
 ] 

Areek Zillur commented on LUCENE-5889:
--

Interesting: Both AnalyzingInfixSuggester  BlendedInfixSuggester implements 
Closeable and on close closes the writer and searchmanager it uses. But looking 
into the Solr factories and suggest component, there seems to be no way to 
close a suggester (close() is never callled). What might happen is, as all the 
failing tests re-uses the same 'solrconfig-phrasesuggest.xml', the 
*InfixSuggester never closes the writer or the searchmanager and on subsequent 
builds throws the LockObtainFailedException?  

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-22 Thread Hoss Man (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14107497#comment-14107497
 ] 

Hoss Man commented on LUCENE-5889:
--

bq. But looking into the Solr factories and suggest component, there seems to 
be no way to close a suggester (close() is never callled).

Huh?

Awe, damn it ... on trunk Suggester registers a CloseHook for the Lookup, but 
apparently that never got backported to 4x.

So it looks like this is in fact a bug in Solr's Suggester class and we've just 
ben (un)lucky that are tests never triggered it, but the new logic added here 
(aparently) shifts when the write locks are held, so w/o the CloseHook the 
tests are failing.

I'll open a new Jira

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-21 Thread Michael McCandless (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14106187#comment-14106187
 ] 

Michael McCandless commented on LUCENE-5889:


Thanks Varun, looks good, I'll commit.  I'll just fix whitespace: if( should 
be if (

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-21 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14106207#comment-14106207
 ] 

ASF subversion and git services commented on LUCENE-5889:
-

Commit 1619635 from [~mikemccand] in branch 'dev/trunk'
[ https://svn.apache.org/r1619635 ]

LUCENE-5889: add commit method to AnalyzingInfixSuggester

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-21 Thread ASF subversion and git services (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14106209#comment-14106209
 ] 

ASF subversion and git services commented on LUCENE-5889:
-

Commit 1619636 from [~mikemccand] in branch 'dev/branches/branch_4x'
[ https://svn.apache.org/r1619636 ]

LUCENE-5889: add commit method to AnalyzingInfixSuggester

 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Fix For: 5.0, 4.11

 Attachments: LUCENE-5889.patch, LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] [Commented] (LUCENE-5889) AnalyzingInfixSuggester should expose commit()

2014-08-20 Thread Michael McCandless (JIRA)

[ 
https://issues.apache.org/jira/browse/LUCENE-5889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14104098#comment-14104098
 ] 

Michael McCandless commented on LUCENE-5889:


Thanks Varun!

I think we shouldn't add another ctor?  (The API is experimental.).
Just add the new commitOnBuild param ...

I think it's fine to init the writer in add/update, but can you pull
that out to its own method, e.g. getWriter?  And I guess it should be
sync'd, in case multiple threads try to add/update at once?

And maybe add some tests showing that you can just use .add and then
refresh and see the suggestions?


 AnalyzingInfixSuggester should expose commit()
 --

 Key: LUCENE-5889
 URL: https://issues.apache.org/jira/browse/LUCENE-5889
 Project: Lucene - Core
  Issue Type: Improvement
  Components: modules/spellchecker
Reporter: Mike Sokolov
 Attachments: LUCENE-5889.patch


 There is no way short of close() for a user of AnalyzingInfixSuggester to 
 cause it to commit() its underlying index: only refresh() is provided.  But 
 callers might want to ensure the index is flushed to disk without closing.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org