[ https://issues.apache.org/jira/browse/SOLR-9616?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15622385#comment-15622385 ]
ASF GitHub Bot commented on SOLR-9616: -------------------------------------- GitHub user timohund opened a pull request: https://github.com/apache/lucene-solr/pull/106 SOLR-9616 Return empty result, when expand component is used with empty result set. This pull request: * Add's early return in expand component, when there is nothing to expand * Add's a regression test for SOLR-9616 Since i am very new to the code i am not sure if this has any other side effects, by checking the other tests it looks good for me, but help is appreciated. You can merge this pull request into a Git repository by running: $ git pull https://github.com/timohund/lucene-solr SOLR-9616 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/lucene-solr/pull/106.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 #106 ---- commit 692dec547d22b366099fc328507a847f1e0cf2c8 Author: Timo Schmidt <timo.h...@yahoo.com> Date: 2016-10-31T13:53:52Z SOLR-9616 Return empty result, when expand component is used with empty result set. * Add's early return in expand component, when there is nothing to expand * Add's a regression test for SOLR-9616 ---- > Solr throws exception when expand=true on empty result > ------------------------------------------------------ > > Key: SOLR-9616 > URL: https://issues.apache.org/jira/browse/SOLR-9616 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Affects Versions: 6.2.1 > Reporter: Timo Hund > Priority: Critical > Fix For: 6.2.1 > > > When i run a query with expand=true with field collapsing and the result set > is empty an exception is thrown: > solr:8984/solr/core_en/select?&fq={!collapse > field=pid}&expand=true&expand.rows=10 > Produces: > "error":{ > "msg":"Index: 0, Size: 0", > "trace":"java.lang.IndexOutOfBoundsException: Index: 0, Size: 0\n\tat > java.util.ArrayList.rangeCheck(ArrayList.java:653)\n\tat > java.util.ArrayList.get(ArrayList.java:429)\n\tat > java.util.Collections$UnmodifiableList.get(Collections.java:1309)\n\tat > org.apache.solr.handler.component.ExpandComponent.process(ExpandComponent.java:269)\n\tat > > org.apache.solr.handler.component.SearchHandler.handleRequestBody(SearchHandler.java:293)\n\tat > > org.apache.solr.handler.RequestHandlerBase.handleRequest(RequestHandlerBase.java:156)\n\tat > org.apache.solr.core.SolrCore.execute(SolrCore.java:2036)\n\tat > org.apache.solr.servlet.HttpSolrCall.execute(HttpSolrCall.java:657)\n\tat > org.apache.solr.servlet.HttpSolrCall.call(HttpSolrCall.java:464)\n\tat > org.apache.solr.servlet.SolrDispatchFilter.doFilter(SolrDispatchFilter.java:257)\n\tat > > org.apache.solr.servlet.SolrDispatchFilter.doFilter(SolrDispatchFilter.java:208)\n\tat > > org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1668)\n\tat > > org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:581)\n\tat > > org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:143)\n\tat > > org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:548)\n\tat > > org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:226)\n\tat > > org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1160)\n\tat > > org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:511)\n\tat > > org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:185)\n\tat > > org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1092)\n\tat > > org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)\n\tat > > org.eclipse.jetty.server.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:213)\n\tat > > org.eclipse.jetty.server.handler.HandlerCollection.handle(HandlerCollection.java:119)\n\tat > > org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:134)\n\tat > org.eclipse.jetty.server.Server.handle(Server.java:518)\n\tat > org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:308)\n\tat > org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:244)\n\tat > > org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:273)\n\tat > org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:95)\n\tat > org.eclipse.jetty.io.SelectChannelEndPoint$2.run(SelectChannelEndPoint.java:93)\n\tat > > org.eclipse.jetty.util.thread.strategy.ExecuteProduceConsume.produceAndRun(ExecuteProduceConsume.java:246)\n\tat > > org.eclipse.jetty.util.thread.strategy.ExecuteProduceConsume.run(ExecuteProduceConsume.java:156)\n\tat > > org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:654)\n\tat > > org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:572)\n\tat > java.lang.Thread.run(Thread.java:745)\n", > "code":500}} > Instead i would assume to get an empty result. > Is this a bug? -- 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