[ 
https://issues.apache.org/jira/browse/GIRAPH-891?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Maja Kabiljo resolved GIRAPH-891.
---------------------------------

    Resolution: Fixed

>  Make MessageStoreFactory configurable
> --------------------------------------
>
>                 Key: GIRAPH-891
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-891
>             Project: Giraph
>          Issue Type: Task
>          Components: bsp
>    Affects Versions: 1.1.0
>            Reporter: Rohan Karwa
>            Priority: Minor
>         Attachments: diff_2ndmay.txt
>
>
> As mentioned in the task: 4213037
> Related to the Task: 
> https://our.intern.facebook.com/intern/tasks/?s=754080093&t=4213037
> Changed the MessageStoreFactory interface and added a new method 
> "initialize(service, config)" in order to do the setting up of message store 
> factory. In order to invoke the class (as per the configuration) on run time, 
> reflection is used for the class invocation and then initialize() method is 
> called on the instance.
> There were few classes which had private factory implementation, for these 
> classes there is no need to implement the initialize() method as the class is 
> never exposed and can't be passed as a configuration.
> Also, in order to make the DiskBackedMessageStore.class value to be passed 
> from the configuration, I moved the implementation to the new file and made 
> this class as a public class. 
> Modified a test case for testing this configuration(DiskBased/InMemory Backed 
> Message Stores). 
> Review Link: https://reviews.apache.org/r/20915/



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to