Skip to content

Commit

Permalink
[FLINK-11287] [rocksdb] RocksDBListState should be using registered s…
Browse files Browse the repository at this point in the history
…erializer in state meta infos

This closes apache#7434.
  • Loading branch information
tzulitai committed Jan 9, 2019
1 parent 1e2aa8e commit e5ed8c8
Showing 1 changed file with 3 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
package org.apache.flink.contrib.streaming.state;

import org.apache.flink.api.common.state.ListState;
import org.apache.flink.api.common.state.ListStateDescriptor;
import org.apache.flink.api.common.state.State;
import org.apache.flink.api.common.state.StateDescriptor;
import org.apache.flink.api.common.typeutils.TypeSerializer;
Expand Down Expand Up @@ -77,19 +76,19 @@ class RocksDBListState<K, N, V>
* @param namespaceSerializer The serializer for the namespace.
* @param valueSerializer The serializer for the state.
* @param defaultValue The default value for the state.
* @param elementSerializer The serializer for elements of the list state.
* @param backend The backend for which this state is bind to.
*/
private RocksDBListState(
ColumnFamilyHandle columnFamily,
TypeSerializer<N> namespaceSerializer,
TypeSerializer<List<V>> valueSerializer,
List<V> defaultValue,
TypeSerializer<V> elementSerializer,
RocksDBKeyedStateBackend<K> backend) {

super(columnFamily, namespaceSerializer, valueSerializer, defaultValue, backend);
this.elementSerializer = elementSerializer;

ListSerializer<V> castedListSerializer = (ListSerializer<V>) valueSerializer;
this.elementSerializer = castedListSerializer.getElementSerializer();
}

@Override
Expand Down Expand Up @@ -281,7 +280,6 @@ static <E, K, N, SV, S extends State, IS extends S> IS create(
registerResult.f1.getNamespaceSerializer(),
(TypeSerializer<List<E>>) registerResult.f1.getStateSerializer(),
(List<E>) stateDesc.getDefaultValue(),
((ListStateDescriptor<E>) stateDesc).getElementSerializer(),
backend);
}

Expand Down

0 comments on commit e5ed8c8

Please sign in to comment.