Skip to content
This repository has been archived by the owner on Nov 16, 2021. It is now read-only.

[osgi enterprise][jmx] errata and some unclear points #7

Open
bjhargrave opened this issue Apr 18, 2010 · 2 comments
Open

[osgi enterprise][jmx] errata and some unclear points #7

bjhargrave opened this issue Apr 18, 2010 · 2 comments
Assignees
Labels
publiccomment Public comment

Comments

@bjhargrave
Copy link
Member

Original bug ID: BZ#87
From: Dmytro Pishchukhin <dmytro.pishchukhin@gmail.com>
Reported version: R4 V4.2

@bjhargrave
Copy link
Member Author

Comment author: Dmytro Pishchukhin <dmytro.pishchukhin@gmail.com>

Errata:
124.5.6 Boolean and boolean types are missed in property type syntax.
124.8.3.28 P_BOOLEAN type is missed in SCALAR list
124.9.4.6 NAME_ITEM type should be SimpleType.STRING
124.9.5.16 SERVICE_EVENT_TYPE misses BUNDLE_IDENTIFIER type. BUNDLE_IDENTIFIER_ITEM is a part of SERVICE_EVENT_TYPE
124.9.5.25 IllegalArgumentException never throws in listServices() method

Unclear points:

  • BundleEvents and ServiceEvents handling (124.9.2.2, 124.9.5.16). 124.2.3 can contain information how to provide JMX notification (MBean should implement javax.management.NotificationBroadcaster)
  • There is no infomration how to register MBean for every instance of Compendium Service (124.3.2, 124.3.4). What ObjectName key property should be added to Compendium Service OBJECTNAME (",service.id=?")

Some small API improvements:
124.9.3.22, 124.9.3.27 - can throw IllegalArgumentException if the bundle indicated does not exist
124.9.3.21, 124.9.3.28, 124.9.3.41 - can throw IllegalArgumentException if locations/bundleIdentifiers and urls/newlevels arrays are null or have different lenght
124.13.1.22 - can throw IllegalArgumentException if the groupname is not a Group
124.13.1.32, 124.13.1.38, 124.13.1.40 - can throw IllegalArgumentException if the filter is invalid
124.13.1.46 - can throw IllegalArgumentException if the groupname is not a Group
124.13.1.50 - can throw IllegalArgumentException if the username is not a User

@bjhargrave
Copy link
Member Author

Comment author: @bosschaert

Needs to be taken into account in the RFC 169 work.

@bjhargrave bjhargrave added the publiccomment Public comment label May 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
publiccomment Public comment
Projects
None yet
Development

No branches or pull requests

2 participants