Mike,
I will raise new issues as per your comments. There are more comments (the
first set actually) in an e-mail that I sent before the one you acted on.
See [1]
[1] http://lists.oasis-open.org/archives/sca-assembly/200902/msg00005.html
++Vamsi
Apache Tuscany Committer http://tuscany.apache.org
Apache Geronimo Committer and Member of PMC http://geronimo.apache.org
Mike Edwards
<mike_edwards@uk.
ibm.com> To
"OASIS Assembly"
10/02/2009 22:47 <>
cc
Subject
Re: [sca-assembly] Some review
comments on
sca-assembly-1.1-spec-cd02-Rev3.pdf
Vamsi,
I'm sorry for missing your comments ahead of the call. I've had a proper
look at them now and I've
acted on them as described inline...
I incorporated the changes into an updated Rev4 along with the 3 editorial
fixes from the TC call today.
If anyone has any problems with these editorial fixes, please shout now!
Yours, Mike.
Strategist - Emerging Technologies, SCA & SDO.
Co Chair OASIS SCA Assembly TC.
IBM Hursley Park, Mail Point 146, Winchester, SO21 2JN, Great Britain.
Phone & FAX: +44-1962-818014 Mobile: +44-7802-467431
Email:
From: C Vamsi <>
To: C Vamsi <>
Cc:
Date: 04/02/2009 20:55
Subject: Re: [sca-assembly] Some review comments on
sca-assembly-1.1-spec-cd02-Rev3.pdf
Some more comments:
Line 870: 4.3.1 instead of 5.3.1?
Updated as an editorial fix
Section 4.4 name should be Component Property?
No - the whole section is "Component" and this is a subheading
Line 1064: @name attribute
Fixed as an editorial
Line 1064-1066: Should the wording be made consistent with that of service
and reference elements?
I think that's a subject for an Issue
Line 1078: change to "Overrides the many specified for this property in the
component type of the implementation."
Fixed as an editorial
Lines 1152-1154: Multiple property elements with same property name is a
violation of "[ASM50030]"
I agree. That's a subject for an issue.
Line 1157: stockQuoteService with a small s
Fixed as an editorial
Line 1166: stockQuoteService with a small s
Fixed as an editorial
++Vamsi
Apache Tuscany Committer http://tuscany.apache.org
Apache Geronimo Committer and Member of PMC http://geronimo.apache.org
C
Vamsi/India/IBM@I
BMIN To
05/02/2009 01:26 cc
Subject
[sca-assembly] Some review comments
on
sca-assembly-1.1-spec-cd02-Rev3.pdf
I have reviewed the document partially and here are some comments on
sca-assembly-1.1-spec-cd02-Rev3.pdf:
Line 212: component with small c
Line 225: us xs:QName for consistency
Line 245: What does implementation 0..1 mean?
Line 377: "zero or more" in bold font.
Line 431-432: Change to "The implementation can use the supplied value in
any way that it chooses."
Section 3.1.4 does not talk about 0..1 for implementation
Line 464: implementation instead of implementationervice
Line 508: tags or annotations?
Should example in 3.3 use @Service annotation as well?
Line 578: Should the componentType have implementation.java tag as it is
computed from the java class?
Line 596: Is it possible to get the default value for the field by
introspection?
Line 600: "Component Implementation" spec instead of "Client and
Implementation" spec?
Line 607-608: Not true for Java EE
Line 609: in xxx.composite is xxx needed?
Section 4.1 name should be Component Implementation?
Section 4.2 name should be Component Service?
Section 4.3 name shoulde Component Reference?
Lines 791-792: replace 2 occurrences of "service" with "reference".
I will post more comments as I review further. I would like to know if I
need to raise NEW ISSUE for some of these.
++Vamsi
Apache Tuscany Committer http://tuscany.apache.org
Apache Geronimo Committer and Member of PMC http://geronimo.apache.org
---------------------------------------------------------------------
To unsubscribe from this mail list, you must leave the OASIS TC that
generates this mail. Follow this link to all your TCs in OASIS at:
https://www.oasis-open.org/apps/org/workgroup/portal/my_workgroups.php
---------------------------------------------------------------------
To unsubscribe from this mail list, you must leave the OASIS TC that
generates this mail. Follow this link to all your TCs in OASIS at:
https://www.oasis-open.org/apps/org/workgroup/portal/my_workgroups.php
Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number
741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
