[GitHub] storm pull request #1811: STORM-1886 Extend KeyValueState iface with delete

[asfgit]

Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] storm pull request #1811: STORM-1886 Extend KeyValueState iface with delete

[kosii]

GitHub user kosii opened a pull request:

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

STORM-1886 Extend KeyValueState iface with delete

The same as #1470, just against the  
[1.x-branch](https://github.com/apache/storm/tree/1.x-branch) 

This pull request incorporates #1606 and #1160, because my patch depends on 
#1606, and #1606 depends on #1160.

Compared to #1470, I had to add a new commit f115162 , because 
[1.x-branch](https://github.com/apache/storm/tree/1.x-branch) uses Java 1.7.

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

$ git pull https://github.com/s4mDev/storm keyvaluestate-with-delete-1.x

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

https://github.com/apache/storm/pull/1811.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 #1811


commit f6f35dd98d2492a38aa4d61da7f6caee4ec2f31a
Author: Alessandro Bellina 
Date:   2016-02-27T04:42:59Z

STORM-1228: port fields_test to java

commit e5ae496e78aaf7a20a05bcff9139e2a6fd1ad551
Author: Alessandro Bellina 
Date:   2016-02-29T23:55:29Z

STORM-1228: code review comments

commit cdf80153009db83527bfbf0e5f3ab61126c5475e
Author: Alessandro Bellina 
Date:   2016-03-02T06:32:37Z

STORM-1228: instantiate fields in each test

commit 6d3a90d53ea425881b8b3c20a52932c7977bbccc
Author: Robert (Bobby) Evans 
Date:   2016-08-04T15:43:02Z

STORM-2020: Stop using sun internal classes.

commit 3d92358ef5a4e1be5dd86d32c960279d5f824f91
Author: Robert (Bobby) Evans 
Date:   2016-08-04T16:24:08Z

Addressed some review comments

commit 44f0beb18fe4b4c304fa73339df02a6abee99445
Author: Robert (Bobby) Evans 
Date:   2016-08-04T16:29:24Z

STORM-2022: update Fields test to match new behavior

commit d55ddffbe6c5bc2c7f022b530e5acfc0bb50c07f
Author: Balazs Kossovics 
Date:   2016-06-02T12:32:06Z

STORM-1886 Extend KeyValueState iface with delete

The patch also provides implementation for delete in
RedisKeyValueState and InMemoryKeyValueState.

commit 2eb308a6d36463b758d1f571cfeb6c9098176d40
Author: Balazs Kossovics 
Date:   2016-06-24T15:51:32Z

Use a Map with tombstones to represent deletions

commit 69068706fd43406a1a8eba63ebbc66932c174cef
Author: Balazs Kossovics 
Date:   2016-11-28T16:24:16Z

STORM-1886 Address last comments

* make tombstone static
* hide the use of Optional from the users
* fix race condition
* make encode and decode methods static

commit f115162f7ae65257d574a729426c845486a057ed
Author: Balazs Kossovics 
Date:   2016-12-02T12:39:06Z

STORM-1886 Make patch compatible with Java 1.7




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---