You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2016/08/02 13:40:20 UTC
[jira] [Commented] (DISPATCH-437) Reconcile C and python management
agents
[ https://issues.apache.org/jira/browse/DISPATCH-437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15403973#comment-15403973 ]
ASF subversion and git services commented on DISPATCH-437:
----------------------------------------------------------
Commit 2bcb2f766a5667767b6aa8ee7b1073f0b5f542fd in qpid-dispatch's branch refs/heads/DISPATCH-437-1 from [~ganeshmurthy]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-dispatch.git;h=2bcb2f7 ]
DISPATCH-437 - Added API around the agent
> Reconcile C and python management agents
> ----------------------------------------
>
> Key: DISPATCH-437
> URL: https://issues.apache.org/jira/browse/DISPATCH-437
> Project: Qpid Dispatch
> Issue Type: Bug
> Components: Management Agent
> Affects Versions: 0.6.0
> Reporter: Alan Conway
> Assignee: Ganesh Murthy
>
> The router now has two management agents, one in C and one in Python. They have overlapping and inconsistent functionality, which creates a difficult user experience. They need to be more closely aligned.
> Issues noted so far:
> - python agent "identity" attribute is unique per-agent. C agent "identity" is only unique per type and cannot be used in READ requests (DISPATCH-409)
> - python agent allows delete by name or identity alone, C agent requires the client specify the type as well (DISPATCH-408)
> - C agent does not do string conversions for integral attribute types as reqiured by the management spec. The "fix" in qdmanage is incorrect (DISPATCH-411)
> - There are two address types - org.apache.qpid.dispatch.router.config.address and org.apache.qpid.dispatch.router.address. There is no shortname for org.apache.qpid.dispatch.router.address. Should short names be removed ?
> I recommend that we need:
> 1. A single code path to validate/convert/insert defaults in incoming requests based on the schema.
> 2. A clear, documented statement of what "identity" means and whether it is type-scoped or agent-scoped, with a re-implementation of either the C or python entities to comply.
> We may need to rework the interface between python and C to make it efficient and thread-safe, or refactor some/all of the python functionality as C, but we do need to get rid of the redundancy and inconsistency.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org