[
https://issues.apache.org/jira/browse/CHAIN-58?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13162683#comment-13162683
]
Simone Tripodi commented on CHAIN-58:
-------------------------------------
Hi Elijah!
I am prototyping on my local repo, after applied your patch, something like:
{code}
interface Command<K, V, C extends Context<K, V>>
{code}
In this way, commands already know wich kind of keys/values and context they
manipulate - of course it implies a lot of modifications in the current code.
If it sounds reasonable to you and you have cycles to provide a patch, it would
be wonderful - I don't have a lot of time to develop it myself during these
months, but I can review patches. Moreover I would like you continue leadering
that idea ;)
Last recommendation: please turn off tabs in your IDE and use spaces instead :P
Many thanks in advance for your hard work!
-Simo
> Update Chain Context interface to use K,V generics
> --------------------------------------------------
>
> Key: CHAIN-58
> URL: https://issues.apache.org/jira/browse/CHAIN-58
> Project: Commons Chain
> Issue Type: Improvement
> Affects Versions: 2.0
> Reporter: Elijah Zupancic
> Fix For: 2.0
>
> Attachments: chain-58.diff
>
>
> As discussed in the mailing list, I am suggesting that we change the
> definition of Context from:
> {noformat}
> public interface Context extends Map<String, Object> {
> {noformat}
> to:
> {noformat}
> public interface Context<K, V> extends Map<K, V> {
> {noformat}
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
