I do not know what to say on this, other then we are stuck between a rock and a hard place. I would say we should just stop using fabricator all together, but then again we already have two tickets opened with ASF-infrastructure not going anywhere for months (moving our site to a CMS, fixing the broken confluence to wiki publishing) so moving the process officially to some ASF review board type thing might have similar issues.
What we might have to do is host phabricator ourselves like we are hosting our unit testing ourselves. I do not know what else to say on this. On Sat, Sep 28, 2013 at 8:55 PM, Sean Busbey <bus...@cloudera.com> wrote: > Bump. Any update on this? > > > On Tue, Sep 17, 2013 at 12:41 PM, Edward Capriolo <edlinuxg...@gmail.com > >wrote: > > > I do not like this. It is inconvenience when using a mobile device, but > > more importantly it does not seem very transparent to our end users. For > > example, a user is browsing jira they may want to review the code only on > > review board (not yet attached to the issue), they should not be forced > to > > "sign up" to help in the process. > > > > Would anyone from facebook care to chime in here? I think we all like > > fabricator for the most part. Our docs suggest this fabricator is our > > de-facto review system. As an ASF project I do not think requiring a > login > > on some external service even to review a jira is correct. > > > > > > On Tue, Sep 17, 2013 at 12:27 PM, Xuefu Zhang <xzh...@cloudera.com> > wrote: > > > > > Yeah. I used to be able to view w/o login, but now I am not. > > > > > > > > > On Tue, Sep 17, 2013 at 7:27 AM, Brock Noland <br...@cloudera.com> > > wrote: > > > > > > > Personally I prefer Review Board. > > > > > > > > On Tue, Sep 17, 2013 at 8:31 AM, Edward Capriolo < > > edlinuxg...@gmail.com> > > > > wrote: > > > > > I never remeber having to log into phabricator to view a patch. Has > > > this > > > > > changed recently? I believe that having to create an external > account > > > to > > > > > view a patch in progress is not something we should be doing. > > > > > > > > > > > > > > > > -- > > > > Apache MRUnit - Unit testing MapReduce - http://mrunit.apache.org > > > > > > > > > > > > > -- > Sean >
