On Thu, Sep 20, 2012 at 6:49 PM, Alex Huang <alex.hu...@citrix.com> wrote: > > All you have to do is look up your name and then look to see if the review > has been changed from yesterday. If you haven't kept track of this daily > email, then you need to look at all the items under your name and work on all > of them. Please scan the list for your name. > > - The first column is the review #. > - The second column is for who to take action. > - The third column is the version this patch should go into. If it is 4+, > you don't have to take action before 4.0 release. > - The fourth column is if the review has changed since last push. If you're > following this email daily, you should be able to just monitor this column to > determine if there's new action item for you since the last push. > - The fifth column is what I believe is the action needed to close out this > review. > > 5655 Hugo 4+ Respond to Review > 5806 Deepti/Nitin 4 Nitin to respond to diff and commit > if okay to ship > 6473 Jamshid 4 Mark as submit > 6492 Krishna 4+ Respond to review comments > 6523 Nitin 4+ Respond to Rohit > 6614 Krishna 4+ Respond to review. > 6702 Vijay 4 Responde to review > 6733 Gregg 4 Mark as submit > 6781 Alena 4+ Respond to questions > 6881 Jie 4 Mark as submit > 6928 Sateesh 4 Commit it > 6937 Pradeep 4 Review > 6941 Pradeep 4 Review and Commit > 7013 Pradeep 4 Respond to comment > 7018 Nitin 4 Commit it > 7052 Prasanna/Frank 4 Commit it
Alex / Prasanna / Pradeep / Frank - I think (and put this in the review) that the patch in review 7052 isn't needed. See my notes in the review itself for reasoning. > 7079 David N 4 Review and Commit > 7083 Pradeep 4 Resubmit w/ changes from review > 7099 4+ > 7106 Jessica T 4 Review and Commit > 7109 Jessica T 4 Review and Commit > 7114 Jessica T 4 Review and Commit > 7119 David N 4 Review and Commit > 7125 Anthony 4 Y Review and commit > 7130 Nitin 4 Y Review and commit > 7165 4+ Y > 7168 Kishan 4 Y Review and Commit > 7187 4+ Y > 7188 Prasanna 4 Y Mark as submit > 7191 Kelven 4 Y Determine if it's needed for 4.0. Review and > Commit > 7192 4+ Y > 7195 Marcus 4 Y Mark as submit >
