[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [sca-assembly] Some review comments on sca-assembly-1.1-spec-cd02-Rev3.pdf
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 <sca-assembly@lists.oasis-open.org> 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: mike_edwards@uk.ibm.com From: C Vamsi <vamsic007@in.ibm.com> To: C Vamsi <vamsic007@in.ibm.com> Cc: sca-assembly@lists.oasis-open.org 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 sca-assembly@lists.oasis-open.org 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
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]