[ 
https://issues.apache.org/jira/browse/USERGRID-254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14243079#comment-14243079
 ] 

ASF GitHub Bot commented on USERGRID-254:
-----------------------------------------

Github user tnine commented on a diff in the pull request:

    https://github.com/apache/incubator-usergrid/pull/120#discussion_r21706204
  
    --- Diff: 
stack/core/src/main/java/org/apache/usergrid/corepersistence/CpSetup.java ---
    @@ -82,11 +78,9 @@ public CpSetup( EntityManagerFactory emf, 
CassandraService cass ) {
         }
     
     
    -
    -
         public static Injector getInjector() {
             if ( injector == null ) {
    -            injector = Guice.createInjector( new GuiceModule() ); 
    +            injector = Guice.createInjector( new GuiceModule( emf ) ); 
    --- End diff --
    
    Do we need to do this here?  This seems like it's creating a circular 
dependency of the EMF into Guice, but the EMF depends on Guice.


> Add Event system to Collections module, use it to clean up old indices
> ----------------------------------------------------------------------
>
>                 Key: USERGRID-254
>                 URL: https://issues.apache.org/jira/browse/USERGRID-254
>             Project: Usergrid
>          Issue Type: Story
>            Reporter: Todd Nine
>            Assignee: David Johnson
>
> When the Core module EntityManager deletes or updates Entities in the 
> Collections module, the Query Index records for the old versions of that 
> Entity need to be deleted.
> To enable this, we added an eventing system to the Collections module so that 
> callers can hook into entity create, update and delete events.
> And we added event handlers to clean up the Query Index records for those 
> events. 
> See the new package in Collections module:
>    org.apache.usergrid.persistence.collection.event
> And in Core:
>    org.apache.usergrid.corepersistence.events



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to