[GitHub] flink pull request #5365: [FLINK-8515] update RocksDBMapState to replace dep...

2018-03-09 Thread bowenli86
Github user bowenli86 closed the pull request at:

https://github.com/apache/flink/pull/5365


---


[GitHub] flink pull request #5365: [FLINK-8515] update RocksDBMapState to replace dep...

2018-01-25 Thread bowenli86
GitHub user bowenli86 opened a pull request:

https://github.com/apache/flink/pull/5365

[FLINK-8515] update RocksDBMapState to replace deprecated remove() with 
delete()

## What is the purpose of the change

RocksDBMapState is currently using `rocksdb#remove()` which is deprecated. 
Should be replaced with `rocksdb#delete()`

## Brief change log

update RocksDBMapState to replace deprecated remove() with delete()

## Verifying this change

This change is already covered by existing tests, such as 
`StateBackendTestBase#testMapState()`

## Does this pull request potentially affect one of the following parts:

none

## Documentation

none

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

$ git pull https://github.com/bowenli86/flink FLINK-8515

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

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


commit 8cd47d5e6110e29e4c6c36aea08543048010b9aa
Author: Bowen Li 
Date:   2018-01-26T04:25:00Z

[FLINK-8515] update RocksDBMapState to replace deprecated remove() with 
delete()




---