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

ASF GitHub Bot commented on STORM-1246:
---------------------------------------

GitHub user abhishekagarwal87 opened a pull request:

    https://github.com/apache/storm/pull/1117

    STORM-1246: port backtype.storm.local-state to java

    1. Whatever conversion logic was common across multiple files, remain in 
local_state_converter.clj.
    2. Conversion logic exclusively used in a clj is moved to that clj e.g. 
ls-local-assignments conversion has been moved to supervisor.clj. This will 
make it easy to port individual clojure files.
    3. As suggested by Robert in JIRA, I have ported get/set functionality into 
LocalState class.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/abhishekagarwal87/storm local-state

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/1117.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 #1117
    
----
commit 7363a24de22799c6488086cbc66a7b708eb3fdd0
Author: Abhishek Agarwal <abhishek.agar...@inmobi.com>
Date:   2016-02-17T17:29:15Z

    STORM-1246: port backtype.storm.local-state to java

commit 35a0052ca23c153a8818707770ad5e871829389f
Author: Abhishek Agarwal <abhishek.agar...@inmobi.com>
Date:   2016-02-17T17:36:40Z

    STORM-1246: Remove extra line

----


> port backtype.storm.local-state to java
> ---------------------------------------
>
>                 Key: STORM-1246
>                 URL: https://issues.apache.org/jira/browse/STORM-1246
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: Abhishek Agarwal
>              Labels: java-migration, jstorm-merger
>
> Wrapper around LocalState, with some helper functions for converting between 
> storm and thrift.



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

Reply via email to