Hi kaka

You're correct, these comments are not correct for RocksDBMapState now, will 
correct it with a hotfix.

Best
Yun Tang
________________________________
From: kaka chen <kaka11.c...@gmail.com>
Sent: Friday, July 5, 2019 15:05
To: dev@flink.apache.org
Subject: A question about RocksDBListState and RocksDBMapState.

Hi All,

I noticed RocksDBListState and RocksDBMapState have both the following
comments:

* <p>{@link RocksDBStateBackend} must ensure that we set the
* {@link org.rocksdb.StringAppendOperator} on the column family that
we use for our state since
* we use the {@code merge()} call.


However, from the source code, I found only RocksDBListState use
rocksdb::merge() function which used StringAppendTestOperator in rocksdb to
merge the list entries when compaction or getting key. I don't know why
RocksDBMapState is related with these comments, could someone to explain
it, thanks.

Thanks,
Kaka

Reply via email to