[jira] [Resolved] (GIRAPH-891) Make MessageStoreFactory configurable

["Maja Kabiljo (JIRA)" <ji...@apache.org>]

     [ 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)