[
https://issues.apache.org/jira/browse/BOOKKEEPER-205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13508661#comment-13508661
]
Jiannan Wang commented on BOOKKEEPER-205:
-----------------------------------------
I'm not sure about this patch. It creates another variant for tests, which will
increase the test time by 50%.
----
Yeah, it's a problem. I don't know if it's possible to specify
LedgerManagerFactory classes in Maven command parameter to improve it.
Actually, we only test for HierarchicalLedgerManagerFactory and
MSLedgerManagerFactory in our inner usage.
In terms of code, generally it looks fine. I'd rename MetastoreClass to
MetastoreImplementationClass or MetastoreImplClass. Also, the javadoc should
mention that the configuration options are only used it Metastore ledger
manager is enabled in the other config items.
-------
Will change.
Also, there's a failing test in the precommit build, and something went in
recently which is messing with the patch application.
-------
I'll check it.
> implement a MetaStore based ledger manager for bookkeeper client.
> -----------------------------------------------------------------
>
> Key: BOOKKEEPER-205
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-205
> Project: Bookkeeper
> Issue Type: Sub-task
> Components: bookkeeper-client
> Reporter: Sijie Guo
> Assignee: Jiannan Wang
> Fix For: 4.2.0
>
> Attachments: BOOKKEEPER-205.diff
>
>
> implement a MetaStore based ledger manager for bookkeeper client.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
