[
https://issues.apache.org/jira/browse/EAGLE-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15429517#comment-15429517
]
ASF GitHub Bot commented on EAGLE-487:
--------------------------------------
GitHub user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/367
use guice module better in app framework
EAGLE-487
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/yonzhang/incubator-eagle develop
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-eagle/pull/367.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #367
----
commit 1f708cf670892833f8d9dfafcbf251feabd98773
Author: yonzhang <[email protected]>
Date: 2016-08-19T17:26:11Z
eagle-data-process project clean up to contain only common processing
classes
commit d6f3b7b7062d7cf6a0b0a68e7f8f04c30599d0ee
Author: yonzhang <[email protected]>
Date: 2016-08-20T21:21:29Z
guice module bug fix and policy code cleanup
commit cf97ab60d1b62de0de1198857d4ad48ede319035
Author: yonzhang <[email protected]>
Date: 2016-08-20T21:24:08Z
fix merge conflicts
----
> use guice module better in app framework
> ----------------------------------------
>
> Key: EAGLE-487
> URL: https://issues.apache.org/jira/browse/EAGLE-487
> Project: Eagle
> Issue Type: Improvement
> Affects Versions: v0.5.0
> Reporter: Edward Zhang
> Assignee: Edward Zhang
> Fix For: v0.5.0
>
>
> There are multiple issues with using guice module in app framework.
> 1. ApplicationProviderServiceImpl is instantiated twice
> 2. the interface between application and eagle server is not clear. It is not
> correct to only let application to register submodules,, instead it should
> includes some state inherited from eagle server, for example metadata storage
> type.
> 3. not necessary to use module scope, it is pretty over-engineering when
> application only want to register some module.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
