Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sofaboot 4.0 support #93

Merged
merged 8 commits into from
Aug 28, 2023
Merged

Sofaboot 4.0 support #93

merged 8 commits into from
Aug 28, 2023

Conversation

Lo1nt
Copy link
Collaborator

@Lo1nt Lo1nt commented Aug 8, 2023

针对 Throwable, Atomic* 类, 在jdk17下避免通过反射进行序列化与反序列化

Lo1nt and others added 3 commits June 19, 2023 11:16
* feat: atomic

* feat: ignore failure when set accessible

* feat: replace reflection with specific get set for atomic

* fix: remove handle

* fix: log level

* feat: stack trace serialize without reflect

* add test case for stack trace

* fix: log

* feat: throw v1

* feat: throwable

* format

* format

* chore: cr fix

* chore: format

* remove version specific

* version

* chore: add ci on sofaboot-4.0-support

---------

Co-authored-by: lo1nt <[email protected]>
* feat: atomic

* feat: ignore failure when set accessible

* feat: replace reflection with specific get set for atomic-s

* fix: remove handle

* fix: log level

* feat: stack trace serialize without reflect

* add test case for stack trace

* fix: log

* feat: throw v1

* feat: throwable

* format

* format

* chore: cr fix

* chore: format

* add ci fo jdk17

* update jdk

* remove version specific

* version

* chore: add ci on sofaboot-4.0-support

* 3.5.0.beta.1 release

---------

Co-authored-by: 均源 <[email protected]>
* feat: atomic

* feat: ignore failure when set accessible

* feat: replace reflection with specific get set for atomic-s

* fix: remove handle

* fix: log level

* feat: stack trace serialize without reflect

* add test case for stack trace

* fix: log

* feat: throw v1

* feat: throwable

* format

* format

* chore: cr fix

* chore: format

* add ci fo jdk17

* update jdk

* remove version specific

* version

* chore: add ci on sofaboot-4.0-support

* 3.5.0.beta.1 release

* remove aci on sofaboot-4.0-support

---------

Co-authored-by: 均源 <[email protected]>
@codecov
Copy link

codecov bot commented Aug 8, 2023

Codecov Report

Merging #93 (7744812) into 3.x (39b1dcb) will increase coverage by 1.97%.
The diff coverage is 60.65%.

@@            Coverage Diff             @@
##              3.x      #93      +/-   ##
==========================================
+ Coverage   26.35%   28.33%   +1.97%     
==========================================
  Files         158      171      +13     
  Lines       11131    11727     +596     
  Branches     1981     2115     +134     
==========================================
+ Hits         2934     3323     +389     
- Misses       7895     8062     +167     
- Partials      302      342      +40     
Files Changed Coverage Δ
...n/java/com/caucho/hessian/util/ReflectionUtil.java 23.25% <23.25%> (ø)
...ain/java/com/caucho/hessian/io/JavaSerializer.java 66.90% <33.33%> (ø)
...ho/hessian/io/throwable/ThrowableDeserializer.java 37.34% <37.34%> (ø)
...m/caucho/hessian/io/throwable/ThrowableHelper.java 41.17% <41.17%> (ø)
...an/io/throwable/StackTraceElementDeserializer.java 58.49% <58.49%> (ø)
...ter/EnumConstantNotPresentExceptionSerializer.java 63.63% <63.63%> (ø)
.../java/com/caucho/hessian/io/SerializerFactory.java 83.12% <65.71%> (-1.50%) ⬇️
...ucho/hessian/io/throwable/ThrowableSerializer.java 65.85% <65.85%> (ø)
...sian/io/throwable/StackTraceElementSerializer.java 66.66% <66.66%> (ø)
...cho/hessian/io/AbstractFieldAdaptorSerializer.java 73.17% <73.17%> (ø)
... and 8 more

... and 6 files with indirect coverage changes

@Lo1nt Lo1nt requested a review from OrezzerO August 8, 2023 07:11
Copy link

@EvenLjj EvenLjj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Lo1nt and others added 5 commits August 14, 2023 15:44
* feat: atomic

* feat: ignore failure when set accessible

* feat: replace reflection with specific get set for atomic-s

* fix: remove handle

* fix: log level

* feat: stack trace serialize without reflect

* add test case for stack trace

* fix: log

* feat: throw v1

* feat: throwable

* format

* format

* chore: cr fix

* chore: format

* add ci fo jdk17

* update jdk

* remove version specific

* version

* chore: add ci on sofaboot-4.0-support

* 3.5.0.beta.1 release

* remove aci on sofaboot-4.0-support

* feat: add support on java util currency

* format

* init 3.5 snapshot

* optimized code (#94)

Co-authored-by: liujianjun.ljj <[email protected]>

* format

* fix: ref problem on currency

* format

* fix: ref bug on atomic

* version snapshot

* enhance: ref

* fix: test

---------

Co-authored-by: 均源 <[email protected]>
Co-authored-by: evenliu <[email protected]>
Co-authored-by: liujianjun.ljj <[email protected]>
* feat: atomic

* feat: ignore failure when set accessible

* feat: replace reflection with specific get set for atomic-s

* fix: remove handle

* fix: log level

* feat: stack trace serialize without reflect

* add test case for stack trace

* fix: log

* feat: throw v1

* feat: throwable

* format

* format

* chore: cr fix

* chore: format

* add ci fo jdk17

* update jdk

* remove version specific

* version

* chore: add ci on sofaboot-4.0-support

* 3.5.0.beta.1 release

* remove aci on sofaboot-4.0-support

* feat: add support on java util currency

* format

* init 3.5 snapshot

* optimized code (#94)

Co-authored-by: liujianjun.ljj <[email protected]>

* format

* fix: ref problem on currency

* format

* enhance: ref

* fix: test

* fix: test exception

* fix: curency test

---------

Co-authored-by: 均源 <[email protected]>
Co-authored-by: evenliu <[email protected]>
Co-authored-by: liujianjun.ljj <[email protected]>
Co-authored-by: lo1nt <[email protected]>
@Lo1nt Lo1nt merged commit 691b4f2 into 3.x Aug 28, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants