realtimetech-solution / opack Goto Github PK

기능 설명

Json Encoding/Decoding 방식 통일화

필요 이유

현재 Json Encoding은 Recursive 방식으로 처리하고 있고, Decoding은 Stack 방식으로 처리하고 있어서 통일화 할 필요가 있다

기타

기능 설명

Opacker에서 serialize/deserialize 진행 시, 대상 Object를 clone해서 관리해야 함.

필요 이유

Object는 참조형이기 때문에, 다른 thread에서 접근할 수 있다. 따라서 serialize/deserialize 진행 중에 접근할 수 없도록, 진행할 Object는 clone을 하여야 한다.

기타

Check the link below
https://www.reddit.com/r/java/comments/u4ytxu/github_realtimetechsolutionopack_fast_object_or/

• There are a lot of reflections / dynamic object allocations / boxing operations. This seems inappropriate for many "real-time" systems (is that a goal or does that just happen to be the company name?)

• It does not seem to address schema evolution, but I might have missed something
  : True.

• Lots of if / else-if checks for checking sequential integers? Why not use a switch?
  : Expecting somewhat low-level optimizer will work on this.

• The used big-endian encoding is more difficult to be optimized later on

버그 설명

List 객체가 Field Transform을 통해 OpackArray로 serialize된 후, Opacker.prepareObjectSerialize() 안에서 element들이 serialize되기 전에 deep clone이 되는 문제.

(serialize된 결과를 확인해보면 List에 serialize가 되지 않은 빈 element들만 추가되어 있다.)

재현 절차

1. 임의의 Class를 만들고, WrapListTransformer가 설정된 List field를 추가한다.
2. List에 OpackValue가 아닌 임의의 객체를 element로 추가한다.
3. Opacker를 통해 (1)의 Class의 인스턴스를 serialize, deserialize한다.

• Serialize 결과
  OpackArray 안에 빈 OpackObject가 있음을 확인할 수 있다.

OpackObject({wrappedTypeList=OpackArray([null,test 1,OpackObject({type=com.realtimetech.opack.test.opacker.WrapListElementTest$TestElement, value=OpackObject({})})])})

오류 로그

com.realtimetech.opack.exception.DeserializeException: Can't set intValue field in TestElement
at com.realtimetech.opack.Opacker.executeDeserializeStack(Opacker.java:408)
at com.realtimetech.opack.Opacker.deserialize(Opacker.java:268)
at com.realtimetech.opack.test.opacker.WrapListElementTest.common(WrapListElementTest.java:81)
at com.realtimetech.opack.test.opacker.WrapListElementTest.testWithWrapListTransformer(WrapListElementTest.java:65)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1540)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1540)

Caused by: java.lang.IllegalArgumentException: Can not set int field

com.realtimetech.opack.test.opacker.WrapListElementTest$TestElement.intValue to null value
at java.base/jdk.internal.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(UnsafeFieldAccessorImpl.java:167)
at java.base/jdk.internal.reflect.UnsafeFieldAccessorImpl.throwSetIllegalArgumentException(UnsafeFieldAccessorImpl.java:171)
at java.base/jdk.internal.reflect.UnsafeIntegerFieldAccessorImpl.set(UnsafeIntegerFieldAccessorImpl.java:80)
at java.base/java.lang.reflect.Field.set(Field.java:780)
at com.realtimetech.opack.compile.ClassInfo$FieldInfo.set(ClassInfo.java:79)
at com.realtimetech.opack.Opacker.executeDeserializeStack(Opacker.java:406)
... 93 more

버그 설명

Opacker를 통해 Null element가 들어간 배열을 serialize할 시, NullPointerException 발생
(ex. String[], Integer[])

재현 절차

1. Example Class에 있는 stringArray에 null element를 추가
2. Example 객체 생성
3. 생성한 Example 객체를 Opacker를 통해 serialize

예상 발생 원인

serialize할 때 array의 element가 null인지 검사를 안함.

오류 로그

Exception in thread "main" java.lang.NullPointerException
at com.realtimetech.opack.Opacker.executeSerializeStack(Opacker.java:150)
at com.realtimetech.opack.Opacker.serialize(Opacker.java:74)
at com.realtimetech.opack.example.Example.main(Example.java:352)

기타