-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hi Bob,
On 15/08/14 14:58, Bob Paulin wrote: > I noticed in https://issues.apache.org/jira/browse/FELIX-3511 Java > Concurrency is being introduced to the code base. A couple of > thoughts on this. > > 1) With this not being backwards compatible with earlier versions > does it make sense to increment at least the minor version (ie 1.3 > -> 1.4). Java 8 introduces a slew of incompatibility with prior > releases with the changes to the collection libraries so I'm > curious how other projects are handling this. As I understand the issue, it talks about going from Java 5 to Java 6 as required version, but yes, strictly speaking this would mean a minor bump at least. We should be more strict on this, I agree (made the same mistake in the Felix HTTP project as well). > 2) Event admin has no tests. I would be interested in working on > creating some tests for this project. Any thoughts on where to > begin with this effort? I think the current implementation is written (and maintained) by people that have access to the OSGi TCK, so they can tests the implementation against the specification, which might explain the lack of unit and integration tests. But to get started: just start writing unit tests against the code in trunk and provide them as patches. It is always good to have tests written by somebody else, as they most often have new/fresh insights in the use cases... > 3) It appears there maybe other areas of the event admin code that > might benefit from the Concurrency objects. Perhaps the use of one > of the Queue constructs for holding some of the asynchronous > events. Thoughts on this? As long as it has positive effect on the performance, I think nobody will complaint about this, if you're willing to provide patches ;) - -- Met vriendelijke groeten | Kind regards Jan Willem Janssen | Software Architect +31 631 765 814 /My world is revolving around INAETICS and Amdatu/ Luminis Technologies B.V. Churchillplein 1 7314 BZ Apeldoorn +31 88 586 46 00 http://www.luminis-technologies.com http://www.luminis.eu KvK (CoC) 09 16 28 93 BTW (VAT) NL8169.78.566.B.01 -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.17 (Darwin) Comment: GPGTools - http://gpgtools.org iQIcBAEBAgAGBQJT7gruAAoJEKF/mP2eHDc4vO8QANxPmNAzlgzUbhxD0VfLN2++ SlFrTqAGxaIS3rI3fzacBn5Z/v12azwyKLE39WfUEak+i5539LrcQYu8b2mOg4ML rtz2Yz6r/IcSDixO5y17HrdHyGle58u/gPzJecQqz5j3iqHi4P8eKtNx8FWrG91c 3X56VsP0sb718YswDnu8WXwy8V8I6p1h5vJdYJRx2DcOml+TEcxHxGTH6h7pDePt DcsPF7E3XaHGWpPvJ5GX8JYjzl80TM5xV4d47VAGdeVCY+JewsC6OlDtxhargGFG O0wEM9RqI2/R9OTB1K6STIwQZvglyjvUp1odgGEvgl8B/kbSh7WHprtUWIrG+D8Z cIEBjS9mYIkPEQQo0NMZIqt1XfEMiK5qElqa7v0z8IDGbhaPHy+vGpK84uhItz6M ++tEZ66OGfNaVJG9ueY+093M/J70V57ylw3Mz1Vhf7yChNa1iuT+sCU0OLOPom4K 4UL3H59m9SCXGzg5EoVEglpggdWZCaQ//73mWp7CLob6Kwk8xCvRtC5yVzbxIAgh UUVcltvSvIDklu9l9k4Di15Q4VyDz9HSu2ZilJJcWln4ZnaanGqxdoRu4tORw7fg jchVs8Bcw1TmSGTjAwzZkKlFoZfGPPkLpa9/SXYDPfsgeU43hqJyazQ7a7ll0SpP 62oji7tQ0LgRHEnbuZcs =PraV -----END PGP SIGNATURE-----
