Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/1950#discussion_r124291484
--- Diff: docs/State-checkpointing.md ---
@@ -70,6 +70,46 @@ json config with the following properties.
}
}
```
+
+For Redis Cluster state this is a json config with the following
properties.
+
+```
+ {
+ "keyClass": "Optional fully qualified class name of the Key type.",
+ "valueClass": "Optional fully qualified class name of the Value type.",
+ "keySerializerClass": "Optional Key serializer implementation class.",
+ "valueSerializerClass": "Optional Value Serializer implementation
class.",
+ "jedisClusterConfig": {
+ "nodes": ["localhost:7379", "localhost:7380", "localhost:7381"],
+ "timeout": 2000,
+ "maxRedirections": 5
+ }
+ }
+```
+
+NOTE: If you used Redis state with Storm version 1.1.0 or earlier, you
would need to also migrate your state since the representation of state has
changed
--- End diff --
I'm trying to address this via adding `version` field to CheckpointState
class, but stuck on migrating old checkpoint spout state to newer. Kyro doesn't
support forward compatibility, and even seemed not be able to read with copied
old class. I have no idea how to do it right now.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
