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 <abell...@yahoo-inc.com> Date: 2016-02-27T04:42:59Z STORM-1228: port fields_test to java commit e5ae496e78aaf7a20a05bcff9139e2a6fd1ad551 Author: Alessandro Bellina <abell...@yahoo-inc.com> Date: 2016-02-29T23:55:29Z STORM-1228: code review comments commit cdf80153009db83527bfbf0e5f3ab61126c5475e Author: Alessandro Bellina <abell...@yahoo-inc.com> Date: 2016-03-02T06:32:37Z STORM-1228: instantiate fields in each test commit 6d3a90d53ea425881b8b3c20a52932c7977bbccc Author: Robert (Bobby) Evans <ev...@yahoo-inc.com> Date: 2016-08-04T15:43:02Z STORM-2020: Stop using sun internal classes. commit 3d92358ef5a4e1be5dd86d32c960279d5f824f91 Author: Robert (Bobby) Evans <ev...@yahoo-inc.com> Date: 2016-08-04T16:24:08Z Addressed some review comments commit 44f0beb18fe4b4c304fa73339df02a6abee99445 Author: Robert (Bobby) Evans <ev...@yahoo-inc.com> Date: 2016-08-04T16:29:24Z STORM-2022: update Fields test to match new behavior commit d55ddffbe6c5bc2c7f022b530e5acfc0bb50c07f Author: Balazs Kossovics <balazs.kossov...@s4m.io> 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 <balazs.kossov...@s4m.io> Date: 2016-06-24T15:51:32Z Use a Map with tombstones to represent deletions commit 69068706fd43406a1a8eba63ebbc66932c174cef Author: Balazs Kossovics <balazs.kossov...@s4m.io> 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 <balazs.kossov...@s4m.io> 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. ---