axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ruwan Linton <ruwan.lin...@gmail.com>
Subject Re: Please revert API changes done as per AXIS2-4465
Date Fri, 21 Aug 2009 06:39:23 GMT
Andreas and all,

This is my personal view...

I agree our model is "commit then review", but it is always good to discuss
any change that affects the existing API or the behaviour. That is what we
follow in synapse as well, if you remember.

Commit then review doesn't imply that you can change the existing behaviours
and let the community review it. No offence on you at all, of course you are
doing a great job for axiom, axis2 and also for synapse.

BTW: can you please add the public method that you have removed preserving
the behaviour of that method? I guess your proposal on the first reply would
work.

Of course +1 for your work.

Thanks,
Ruwan

On Fri, Aug 21, 2009 at 11:40 AM, Sanjiva Weerawarana <sanjiva@opensource.lk
> wrote:

> +1 Andreas; please keep going .. but a little flare-up once in a way is
> always good to wake everyone up from their slumber ;-).
>
> Sanjiva.
>
>
> On Fri, Aug 21, 2009 at 10:39 AM, Hiranya Jayathilaka <
> hiranya911@gmail.com> wrote:
>
>>
>>
>> On Fri, Aug 21, 2009 at 5:54 AM, Deepal jayasinghe <deepalk@gmail.com>wrote:
>>
>>> >
>>> > 2) I think there is no need to start yelling around about "merrily
>>> > chang[ing] APIs", "things are going to become a big mess" and "engage
>>> > the community before making such drastic changes". I did lots of fixes
>>> > in Axiom and Axis2 since Synapse trunk switched from Axis2 SNAPSHOT to
>>> > 1.5, and when Hiranya switched back to SNAPSHOT there were no
>>> > particular issues related to them, except for the issue discussed
>>> > here. I personally take the suggestion that my changes make of Axis2 a
>>> > big mess as an offense. If that is what some people think, I can stop
>>> > immediately to work on Axis2 and Axiom. With respect to engaging the
>>> > community, as mentioned above, the description of AXIS2-4465 provides
>>> > enough information about the reasons for the change and as Dims
>>> > pointed out, we have a commit-and-review policy. Since the change is
>>> > focused, not massive, can easily be reverted and contains an
>>> > appropriate level of Javadoc, I think that the community has
>>> > everything that is needed to review it.
>>> >
>>> You are doing a great work with Axis2 project and as I can see you are
>>> so active in the project, so there is no need to stop what you are doing.
>>
>>
>> +1.
>>
>> Andreas, you have been a great contributor for Axis2 and AXIOM over the
>> past few months and I'm sure the community highly values and appreciates all
>> your contributions. Most of your changes were very good ones and required
>> ones too. It is just this change needs a bit of reworking and cleanup to
>> maintain compatibility with the custom extensions - that's all :)
>>
>> Thanks,
>> Hiranya
>>
>>
>>> >
>>> > 3) The discussion in AXIS2-4465 pointed to the problem that even by
>>> > preserving the public API of AxisServlet, we will probably not be able
>>> > to avoid breaking subclasses. This is caused by the fact that
>>> > AxisServlet doesn't have an extensible design. If somebody has a
>>> > brilliant idea how to get around this problem (other than reverting
>>> > the fix and reopening the 6 issues it is supposed to solve), please
>>> > speak out.
>>> >
>>> I am not going to ask you to revert the patch, but we should not change
>>> the public APIs. This is not the first time that happens, and I those
>>> times I also point them out. Just because we have commit then review, we
>>> do not have time to go though each and every commits (when I was having
>>> time I did that). So as the commiters of the project let's not try to
>>> break the features we have or let's not change public APIs.
>>>
>>> Thanks,
>>> Deepal
>>> > Andreas
>>> >
>>> > [1] http://markmail.org/message/62dixjx3qrqry3yr
>>> > [2] http://svn.apache.org/viewvc?view=rev&revision=744900
>>> >
>>> > On Thu, Aug 20, 2009 at 18:08, Senaka Fernando<senaka@wso2.com> wrote:
>>> >
>>> >> Hi Andreas,
>>> >>
>>> >> Just wondering what you are trying to achieve here. Is this related
to
>>> auto
>>> >> detection of ports as Hiranya pointed out? While I appreciate the
>>> effort
>>> >> you've put into doing something worthwhile, I believe that getting rid
>>> of a
>>> >> public method in a class is not the correct thing to do. I believe
>>> that what
>>> >> you have done here is the addition of a new portion of code. Can we
>>> make the
>>> >> new portion of code optional? And leave the existing logic as it was?
>>> Also,
>>> >> are you planning further changes to this class? if so, it would
>>> perhaps be
>>> >> better to figure out a more elaborate solution, which safeguards both
>>> the
>>> >> existing level of extensibility of this class and also its public API.
>>> >>
>>> >> Thanks,
>>> >> Senaka
>>> >>
>>> >> On Thu, Aug 20, 2009 at 6:14 PM, Hiranya Jayathilaka <
>>> hiranya911@gmail.com>
>>> >> wrote:
>>> >>
>>> >>> Hi Andreas,
>>> >>>
>>> >>> By looking at the code I got the impression that HTTP transport
>>> receivers
>>> >>> should extend the AxisServletListener class for your logic of port
>>> auto
>>> >>> detection to work. Is that correct? What happens if the transport
>>> receivers
>>> >>> used do not extend this class? All request handler methods call
the
>>> >>> preprocessRequest method which in turns run port auto detection.
If
>>> the
>>> >>> transport receivers do not extend AxisServlerListener how is that
>>> handled?
>>> >>>
>>> >>> Thanks,
>>> >>> Hiranya
>>> >>>
>>> >>>
>>> >>> On Thu, Aug 20, 2009 at 6:05 PM, Andreas Veithen
>>> >>> <andreas.veithen@gmail.com> wrote:
>>> >>>
>>> >>>> Afkham,
>>> >>>>
>>> >>>> The only change I see in the public APIs is the disappearance
of the
>>> >>>> initContextRoot method. We can easily fix this be restoring
the
>>> >>>> original initContextRoot method and let the preprocessRequest
method
>>> >>>> call initContextRoot. Do you see any other things to change?
>>> >>>>
>>> >>>> Andreas
>>> >>>>
>>> >>>> On Thu, Aug 20, 2009 at 13:45, Afkham Azeez<afkham@gmail.com>
>>> wrote:
>>> >>>>
>>> >>>>> Yes Dims. However, if everybody continues to merrily change
APIs,
>>> >>>>> making public methods private & so on, things are going
to become a
>>> >>>>> big mess. Axis2 provides public APIs, and those may be having
>>> >>>>> problems, but still they are public APIs. This is why you
have to
>>> be
>>> >>>>> very careful when defining APIs; if you get them wrong,
you may
>>> have
>>> >>>>> to live with it for a long time.
>>> >>>>>
>>> >>>>> Azeez
>>> >>>>>
>>> >>>>> On Thu, Aug 20, 2009 at 11:38 AM, Davanum Srinivas<
>>> davanum@gmail.com>
>>> >>>>> wrote:
>>> >>>>>
>>> >>>>>> Azeez,
>>> >>>>>>
>>> >>>>>> We are still following, commit-then-review right?
>>> >>>>>>
>>> >>>>>> thanks,
>>> >>>>>> dims
>>> >>>>>>
>>> >>>>>> On 08/20/2009 07:33 AM, Afkham Azeez wrote:
>>> >>>>>>
>>> >>>>>>> Hi Andreas,
>>> >>>>>>> The changes you've done to the APIs as per
>>> >>>>>>> https://issues.apache.org/jira/browse/AXIS2-4465
badly breaks
>>> some of
>>> >>>>>>> the projects that depend on Axis2. Please revert
this, and please
>>> >>>>>>> engage the community before making such drastic
changes in the
>>> >>>>>>> future.
>>> >>>>>>>
>>> >>>>>>>
>>> >>>>>
>>> >>>>> --
>>> >>>>> Thanks
>>> >>>>> Afkham Azeez
>>> >>>>>
>>> >>>>> Blog: http://afkham.org
>>> >>>>> Developer Portal: http://www.wso2.org
>>> >>>>> WSAS Blog: http://wso2wsas.blogspot.com
>>> >>>>> Company: http://wso2.com
>>> >>>>> GPG Fingerprint: 643F C2AF EB78 F886 40C9  B2A2 4AE2 C887
665E 0760
>>> >>>>>
>>> >>>>>
>>> >>>
>>> >>> --
>>> >>> Hiranya Jayathilaka
>>> >>> Software Engineer;
>>> >>> WSO2 Inc.;  http://wso2.org
>>> >>> E-mail: hiranya@wso2.com;  Mobile: +94 77 633 3491
>>> >>> Blog: http://techfeast-hiranya.blogspot.com
>>> >>>
>>> >>
>>> >
>>> >
>>>
>>>
>>> --
>>> Thank you!
>>>
>>>
>>> http://blogs.deepal.org
>>> http://deepal.org
>>>
>>>
>>
>>
>> --
>> Hiranya Jayathilaka
>> Software Engineer;
>> WSO2 Inc.;  http://wso2.org
>> E-mail: hiranya@wso2.com;  Mobile: +94 77 633 3491
>> Blog: http://techfeast-hiranya.blogspot.com
>>
>
>
>
> --
> Sanjiva Weerawarana, Ph.D.
> Founder, Director & Chief Scientist; Lanka Software Foundation;
> http://www.opensource.lk/
> Founder, Chairman & CEO; WSO2, Inc.; http://www.wso2.com/
> Member; Apache Software Foundation; http://www.apache.org/
> Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/
>
> Blog: http://sanjiva.weerawarana.org/
>



-- 
Ruwan Linton
Technical Lead & Product Manager; WSO2 ESB; http://wso2.org/esb
WSO2 Inc.; http://wso2.org
email: ruwan@wso2.com; cell: +94 77 341 3097
blog: http://ruwansblog.blogspot.com

Mime
View raw message