Can someone commit the changes to namespacechanges branch ? Thanks, Dharmesh
On Tue, Jul 16, 2013 at 10:09 AM, Dharmesh Kakadia <dhkaka...@gmail.com>wrote: > juberwell and chip > > Sorry. I forgot to mention that this patches are for namespace branch. Is > this the right way to push changes into feature branch for non-committers ? > > Thanks, > Dharmesh > > > On Tue, Jul 16, 2013 at 2:37 AM, John Burwell <jburw...@basho.com> wrote: > >> Chip, >> >> I apologize for my lack clarity. I am only -1 for master merge until the >> community has reviewed the changes and come to consensus on landing the >> change. I am +1 to push the changes into a dedicated feature branch within >> the ASF repo.. >> >> Thanks, >> -John >> >> On Jul 15, 2013, at 4:34 PM, Chip Childers <chip.child...@sungard.com> >> wrote: >> >> > On Mon, Jul 15, 2013 at 04:30:53PM -0400, John Burwell wrote: >> >> Dharnesh, >> >> >> >> I am placing a pre-emptive -1 on this patch merge because I don't feel >> adequate community awareness/coordination regarding the impact of this >> change. In particular, I want to make the sure the timing is appropriate >> since we doing a lot of master->4.2 merges at the moment. This patch will >> increase the difficulty of these merges. Please email send a [MERGE] email >> (e.g. subject "[MERGE] Global Conversion from com.cloud.* to org.apache.*") >> to the list explaining the scope of the change in detail and the potential >> impacts. Once the community arrives at a consensus on timing and impacts, >> I will withdraw my -1. >> >> >> > >> > AFAIK, the idea was to get them into the repo, under the >> > namespacechanges branch. After they are in, that's a good place to go >> > through the conflict resolution, etc... and then do the process you are >> > suggesting. >> > >> >> Thanks, >> >> -John >> >> >> >> On Jul 15, 2013, at 4:20 PM, Dharmesh Kakadia <dhkaka...@gmail.com> >> wrote: >> >> >> >>> Hi, >> >>> >> >>> I have completed the namespace changes to org.apache. A single patch >> was leaving some deleted files behind. I tried to figure out the problem >> for a day, but didn't get any issue. So I have created 3 different patches >> and uploaded to the reviewboard (https://reviews.apache.org/r/12198/). >> >>> >> >>> I have updated the scripts as well as docs(mostly). I have build it >> with all testcases and successfully ran management server. >> >>> >> >>> I will submit whitespace changes patch soon. In mean time if others >> can test other scripts, packaging, specific extension etc. Is that required >> or full test run is good enough ? we can merge changes on 20th accordingly. >> >>> >> >>> Thanks, >> >>> Dharmesh >> >>> >> >>> >> >>> >> >> >> >> >
