New issue
Advanced search Search tips

Issue 3877 link

Starred by 3 users

Issue metadata

Status: Released
Owner: ----
Closed: Feb 2016



Sign in to add a comment

Gerrit query does not return files when passing a combination of search operators

Project Member Reported by bassem.r...@gmail.com, Feb 5 2016

Issue description

*****************************************************************
*****                                                       *****
***** !!!! THIS BUG TRACKER IS FOR GERRIT CODE REVIEW !!!!  *****
*****                                                       *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, CYANOGENMOD,  *****
***** INTERNAL ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.*****
*****                                                       *****
*****   THOSE ISSUES BELONG IN DIFFERENT ISSUE TRACKERS     *****
*****                                                       *****
*****************************************************************

Affected Version:
Gerrit 2.11.6

What steps will reproduce the problem?
1. ssh -p 29418 <gerrit instance URL> gerrit query "<change-id>" --patchsets --files --comments


What is the expected output? What do you see instead?
With Gerrit 2.11.5, the files are returned properly in the JSON output of this query. Instead using Gerrit 2.11.6, no files are returned in the JSON output for the same change.

Please provide any additional information below.
This combination of search operators are used heavily by our users in their CI machinery to decide what tests to run based on the source files changed by the Gerrit change.

 
Project Member

Comment 1 by huga...@gmail.com, Feb 5 2016

Status: Accepted
Project Member

Comment 2 by bassem.r...@gmail.com, Feb 5 2016

After troubleshooting this, the culprit for this regression is this change [1], which was attempting to fix the issue [2]


[1] https://gerrit-review.googlesource.com/#/c/73749/
[2] https://code.google.com/p/gerrit/issues/detail?id=3762
Project Member

Comment 3 by sschube...@gmail.com, Feb 6 2016

Uh, sorry for that. Proposed fix (untested) is at https://gerrit-review.googlesource.com/#/c/74650/.
Project Member

Comment 4 by david.os...@gmail.com, Feb 7 2016

Status: ChangeUnderReview
Project Member

Comment 5 by sschube...@gmail.com, Feb 8 2016

@Bassem: Would you be able to verify the proposed fix?
Project Member

Comment 6 by huga...@gmail.com, Feb 8 2016

I will verify it
Project Member

Comment 7 by huga...@gmail.com, Feb 8 2016

Labels: FixedIn-2.11.7
Status: Submitted
Status: Released

Sign in to add a comment