JanusGraph / JanusGraph/janusgraph

Gryo reader / writer is broken in JanusGraph 1.0.0

Open
#4,087 1 comment 1 reaction 0 assignees View on GitHub
kind/bug/confirmed
Dominant language
Java
Stars
5.8k
Forks
1.2k
Avg merge
13h 53m
Merged PRs (30d)
6

Description

As noted in the changelog: https://docs.janusgraph.org/changelog/#drop-support-for-gryo-messageserializer
Gryo MessageSerializer [has been dropped in TinkerPop 3.6.0](https://tinkerpop.apache.org/docs/3.6.1/upgrade/#_removed_gryo_messageserializers). However, TinkerPop documentation says that Gryo may still be used for file based applications.

For confirmed bugs, please report:

- Version: 1.0.0
- Storage Backend: -
- Mixed Index Backend: -
- Expected Behavior: file is created with `kryo` format.
- Current Behavior: an exception is thrown: `Method threw 'java.lang.IllegalArgumentException' exception.`.
- Steps to Reproduce:
Try to read or write any graph using kryo serialization.
```
g.io("graph.kryo").read().iterate()
g.io("graph.kryo").write().iterate()
```
### Stack Trace (if you have one)

```
Unable to create serializer "org.apache.tinkerpop.shaded.kryo.serializers.FieldSerializer" for class: java.util.concurrent.atomic.AtomicLong
```

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the `g.io("graph.kryo")` read and write operations and review the referenced TinkerPop Gryo removal documentation alongside the stack trace. Trace the reader and writer paths involved in Kryo serialization; done means both operations work without the serializer exception and produce the expected Kryo-format file.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
databases
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.