Getting a "400 cannot create query for index: .." error in conflicts with column

[Khai Do]

Hello.  I just pushed a change[1] that contains a ton of files to our Gerrit.  When I go look at the change in the UI the 'conflict with' column says "400 cannot create query for index: ..".  Does anybody know what's going on there?  Not sure if the 'index' is part of the query or not but I don't see an 'index' predicate in the documentation[2].  I'm wondering if this error is due to bad syntax in the query or if the query is just too long?  I don't see this error on other changes pushed to the same project.

[1] https://review-dev.openstack.org/#/c/5399

[2] https://gerrit-documentation.storage.googleapis.com/Documentation/2.11/user-search.html

[Shawn Pearce]

On Wed, Dec 30, 2015 at 3:33 PM, Khai Do <zaro...@gmail.com> wrote:

Hello.  I just pushed a change[1] that contains a ton of files to our Gerrit.  When I go look at the change in the UI the 'conflict with' column says "400 cannot create query for index: ..".  Does anybody know what's going on there?  Not sure if the 'index' is part of the query or not but I don't see an 'index' predicate in the documentation[2].  I'm wondering if this error is due to bad syntax in the query or if the query is just too long?  I don't see this error on other changes pushed to the same project.

Yes, its the number of files. The conflicts with panel works by making a query that is just:

  status:open AND (file:A OR file:B OR file:C OR file:D ...)

For every file. A ton of files may create a lot of branches in that predicate tree. There is an upper limit on how many predicates are sent to the index, as otherwise the index query falls over and fails in other unexpected ways.

[Dave Borowitz]

That said in 2.12 the error message should be improved by https://gerrit-review.googlesource.com/71580 to say "too many terms in query".

 

--
--
To unsubscribe, email repo-discuss...@googlegroups.com
More info at http://groups.google.com/group/repo-discuss?hl=en

---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[lucamilanesio]

We have the same problem on GerritHub: possibly the change-screen UX should just handle the REST API call in a graceful way and say:

"Conflict information not available: too many files to compare".

Luca.

[Zaro]

Any possibility to back port this bug fix to stable-2.11 branch?

[Doug Kelly]

FYI, this issue has been reported as https://code.google.com/p/gerrit/issues/detail?id=3256

[Doug Kelly]

Also, if this is still happening, perhaps Dave's fix to 2.12 didn't work? I see GerritHub is currently running 2.12-rc2-438, so I would think it would have the change.

[Saša Živkov]

On Wed, Jan 6, 2016 at 9:33 AM, lucamilanesio <luca.mi...@gmail.com> wrote:

We have the same problem on GerritHub: possibly the change-screen UX should just handle the REST API call in a graceful way and say:

"Conflict information not available: too many files to compare".

+1