My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 2013: Watched Changes empty if watching more than one project
1 person starred this issue and may be notified of changes. Back to list
Status:  Released
Owner:  sop@google.com
Closed:  Jul 2013


Sign in to add a comment
 
Reported by icee...@googlemail.com, Jul 16, 2013
************************************************************
***** NOTE: THIS BUG TRACKER IS FOR GERRIT CODE REVIEW *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, INTERNAL *****
***** ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.    *****
***** THOSE ISSUE BELONG IN DIFFERENT ISSUE TRACKERS!  *****
************************************************************

Affected Version:
gerrit master / gerrit-review.googlesource.com
               (2.7-rc2-842-g1d12e2c) 
What steps will reproduce the problem?
1. add more than one project to 'watched projects'
2. goto My -> watched changes
3.

What is the expected output? What do you see instead?
 
Instead of a listr  of changes, the list is empty. If only one project is selected,
the list populated as expected.


Please provide any additional information below.

Jul 17, 2013
#1 icee...@googlemail.com
I did a git bisect:

add86b27cbb236330fc6403b4addde6ba33a177a is the first bad commit
commit add86b27cbb236330fc6403b4addde6ba33a177a
Author: Shawn Pearce <sop@google.com>
Date:   Wed Jun 26 00:04:41 2013 -0600

    Reimplement is:watched to inline filter query trees
    
    Instead of recursively executing the user's watched notifications
    inline the query parse tree as part of the current query.  This allows
    an index such as Lucene to execute the entire set of notification
    filters, avoiding unnecessary joins through the database.
    
    Change-Id: Id55ec1b84aff8bed99db748de7c5f7332c62fbda

It seems the final query is an and-query instead of an or-query.
Jul 17, 2013
Project Member #2 david.pu...@sonymobile.com
(No comment was entered for this change.)
Summary: Watched Changes empty if watching more than one project (was: Wacthed Changes empty if wacthing more than one project)
Jul 17, 2013
#3 sop@google.com
https://gerrit-review.googlesource.com/47952
Status: ChangeUnderReview
Owner: sop@google.com
Jul 18, 2013
Project Member #4 david.pu...@sonymobile.com
(No comment was entered for this change.)
Status: Submitted
Labels: FixedIn-2.8
Dec 9, 2013
Project Member #5 david.pu...@sonymobile.com
(No comment was entered for this change.)
Status: Released
Sign in to add a comment

Powered by Google Project Hosting