Details
-
Improvement
-
Status: In Review
-
High
-
Resolution: Unresolved
-
None
Description
We do not need full serdes capabilities of Kryo, certainly not its ASM/Objenesis dependencies. Examine the interplay of sal-akka-segmented-journal with Kryo and replace it with a simpler interface, which will result in binary-compatible serialization format.
This will allow us to not shade Kryo, significantly reducing the amount of classes we ship.
Attachments
Issue Links
- blocks
-
CONTROLLER-2044 Improve sal-akka-raft serialization protocol
-
- Confirmed
-
- is blocked by
-
CONTROLLER-2071 Switch to our fork of atomix-storage
-
- Resolved
-
Gerrit Reviews
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 106405,16 | Remove Kryo from atomix-storage | master | controller | Status: NEW | +1 | +1 |
| 104722,1 | Remove atomix.utils.Serializer(Builder) | master | controller | Status: MERGED | +2 | +1 |
| 104723,1 | Remove atomix.utils.serializer.serializers | master | controller | Status: MERGED | +2 | +1 |
| 104725,1 | Do not allow recursive Namespace registration | master | controller | Status: MERGED | +2 | +1 |
| 104726,2 | Do not fall back to default Kryo serializers | master | controller | Status: MERGED | +2 | +1 |
| 104727,2 | Ditch dependency on commons-lang3 | master | controller | Status: MERGED | +2 | +1 |
| 104729,1 | Remove Namespace.Builder.nextId() | master | controller | Status: MERGED | +2 | +1 |
| 104730,1 | Remove Namespace.Builder.setCompatible() | master | controller | Status: MERGED | +2 | +1 |
| 104731,1 | Remove Namespace.MAX_BUFFER_SIZE | master | controller | Status: MERGED | +2 | +1 |
| 104732,1 | Remove Namespace.DEFAULT | master | controller | Status: MERGED | +2 | +1 |
| 104733,1 | Remove Namespace.Builder.setRegistrationRequired() | master | controller | Status: MERGED | +2 | +1 |
| 104734,1 | Remove Namespace.friendlyName() | master | controller | Status: MERGED | +2 | +1 |
| 104735,1 | Remove Namespace.size() | master | controller | Status: MERGED | +2 | +1 |
| 104738,1 | Remove Namespace.populate() | master | controller | Status: MERGED | +2 | +1 |
| 104740,8 | Do not leak Kryo from atomix.storage | master | controller | Status: MERGED | +2 | +1 |
| 104742,8 | Remove Namespace.FLOATING_ID | master | controller | Status: MERGED | +2 | +1 |
| 104743,9 | Remove Namespace.RegistrationBlock | master | controller | Status: MERGED | +2 | +1 |
| 104744,10 | Add RegisteredType | master | controller | Status: MERGED | +2 | +1 |
| 104745,9 | Rename Namespace to KryoJournalSerdes | master | controller | Status: MERGED | +2 | +1 |
| 104746,1 | Fixup javadoc | master | controller | Status: MERGED | +2 | +1 |
| 105035,3 | Move atomix-storage to a top-level directory | master | controller | Status: MERGED | +2 | +1 |