Return-Path: X-Original-To: apmail-incubator-directmemory-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-directmemory-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 069F49A7F for ; Sat, 26 Nov 2011 08:55:34 +0000 (UTC) Received: (qmail 79789 invoked by uid 500); 26 Nov 2011 08:55:33 -0000 Delivered-To: apmail-incubator-directmemory-dev-archive@incubator.apache.org Received: (qmail 79764 invoked by uid 500); 26 Nov 2011 08:55:32 -0000 Mailing-List: contact directmemory-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: directmemory-dev@incubator.apache.org Delivered-To: mailing list directmemory-dev@incubator.apache.org Received: (qmail 79750 invoked by uid 99); 26 Nov 2011 08:55:29 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 26 Nov 2011 08:55:29 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=FREEMAIL_FROM,HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of raffaele.p.guidi@gmail.com designates 74.125.82.43 as permitted sender) Received: from [74.125.82.43] (HELO mail-ww0-f43.google.com) (74.125.82.43) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 26 Nov 2011 08:55:23 +0000 Received: by wwo1 with SMTP id 1so7183567wwo.0 for ; Sat, 26 Nov 2011 00:55:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=0APfo1n4chbWc/SKPZpBDV6qIxVzytUuM1pN8vxoGUY=; b=w9f5DW1m32ChkpjCu2m3LRbMH/A0a2+BbRc34L+mj4aoTLS6jzNB//7SW0Grh20zgj lm8u/wl4mfFglLqR1PCB9SrZCr517rC+QwA7J0AhEdFrM18mPDRoGJN6mD+1xXCjXq1G ibVP9guavPnPBZRi54uFI4c4d6ivDqAs5h1h4= MIME-Version: 1.0 Received: by 10.180.7.97 with SMTP id i1mr37877610wia.23.1322297701903; Sat, 26 Nov 2011 00:55:01 -0800 (PST) Received: by 10.216.37.146 with HTTP; Sat, 26 Nov 2011 00:55:01 -0800 (PST) In-Reply-To: References: Date: Sat, 26 Nov 2011 09:55:01 +0100 Message-ID: Subject: Re: refactoring in a branch From: "Raffaele P. Guidi" To: directmemory-dev@incubator.apache.org Content-Type: multipart/alternative; boundary=f46d041555f0cf2d7404b29f6c17 --f46d041555f0cf2d7404b29f6c17 Content-Type: text/plain; charset=ISO-8859-1 >> raffaele github is a private company not a scm tool it's very different ! I'm totally aware of that (believe me) but _that_ is the workflow I'm used to - and the whole community should recognize that the fork->pull workflow is becoming a mindset and should be recognized in his own regardless the private company behind it (that, thanks god, didn't put an IP on it). Hope that the ASF takes this in consideration while moving to the git scm. Ciao, R On Sat, Nov 26, 2011 at 9:29 AM, Olivier Lamy wrote: > yup use agree create a branch and commit on it. > Like a *git* workflow. (raffaele github is a private company not a scm > tool it's very different ! ) > As it everybody will se that. > You can do it now directly in svn (or tru git svn). > > > 2011/11/26 Raffaele P. Guidi : > > Well, being used to the github workflow this feels quite natural. The > > un-natural thing is asking before doing it (just fork, branch and make a > > pull request when you are done). In any case I think jira should be used > to > > track bugs, ask for features and "assign" items, not to obtain an > > authorization to work on things. It shouldn't prevent spontaneous coding > > from committers. > > > > Ciao, > > R > > Il giorno 26/nov/2011 07:55, "Tommaso Teofili" < > tommaso.teofili@gmail.com> > > ha scritto: > > > >> Hi all, > >> > >> yesterday I was doing some code review on DM and my feeling is that we > >> should do some refactoring on a couple of things: > >> - dependencies: I'd like to keep the dependency list size as low as > >> possible, trying to remove the dependencies that are not "critical" > (i.e.: > >> I wouldn't rewrite a new logger slf4j just works nicely) > >> - modularity: I think it'd be good if we could enhance cohesion and > >> modularity of classes; i.e. Cache and CacheService as they have lots of > >> different methods (different contracts / responsibilities) > >> > >> I know those are a kind of high level topics but I think I could try to > >> create a DM branch where I "propose" some dependencies to be removed and > >> also some code refactoring. > >> I'm proposing that instead of opening tinier issues/tasks on Jira since > I > >> think changes would impact lots of classes so it may be hard to > maintain. > >> What do you think? > >> Tommaso > >> > > > > > > -- > Olivier Lamy > Talend: http://coders.talend.com > http://twitter.com/olamy | http://linkedin.com/in/olamy > --f46d041555f0cf2d7404b29f6c17--