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

[WIP]merge develop into master #286

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[WIP]merge develop into master #286

wants to merge 5 commits into from

Conversation

nocvalight
Copy link
Member

Motivation:

Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.

Modification:

Describe the idea and modifications you've done.

Result:

Fixes #.

If there is no issue then describe the changes introduced by this PR.

dbl-x and others added 2 commits February 18, 2023 12:04
* refactor interceptor

* fix cr

* skip SessionRegistry#process when interceptor execute fail

---------

Co-authored-by: linxin <[email protected]>
* refactor interceptor

* fix cr

* refactor session.acceptor module:
1. remove WriteDataProcessor: avoid creating unnecessary objects
2. standardize the use of generic types to avoid code warnings

* refactor session.acceptor module:
1. remove WriteDataProcessor: avoid creating unnecessary objects
2. standardize the use of generic types to avoid code warnings

* skip SessionRegistry#process when interceptor execute fail

* fix code

---------

Co-authored-by: linxin <[email protected]>
Co-authored-by: bjxiaojian <[email protected]>
dbl-x and others added 3 commits March 7, 2023 10:49
* refactor interceptor

* fix cr

* refactor session.acceptor module:
1. remove WriteDataProcessor: avoid creating unnecessary objects
2. standardize the use of generic types to avoid code warnings

* refactor session.acceptor module:
1. remove WriteDataProcessor: avoid creating unnecessary objects
2. standardize the use of generic types to avoid code warnings

* skip SessionRegistry#process when interceptor execute fail

* Add BufferdDataWritingEngine and simplify DataNodeServiceImpl

* remove Req

* fix code

* fix cr

---------

Co-authored-by: linxin <[email protected]>
Co-authored-by: bjxiaojian <[email protected]>
* refactor interceptor

* fix cr

* refactor session.acceptor module:
1. remove WriteDataProcessor: avoid creating unnecessary objects
2. standardize the use of generic types to avoid code warnings

* refactor session.acceptor module:
1. remove WriteDataProcessor: avoid creating unnecessary objects
2. standardize the use of generic types to avoid code warnings

* skip SessionRegistry#process when interceptor execute fail

* Add BufferdDataWritingEngine and simplify DataNodeServiceImpl

* remove Req

* fix code

* fix cr

* SessionRegistryStrategy->ClientRegistrationHook

* remove SyncConfigHandlerStrategy

* move Hook to registry package

* fix testcase

* fix testcase

* spring bean ClientRegistrationHook

---------

Co-authored-by: linxin <[email protected]>
Co-authored-by: bjxiaojian <[email protected]>
* refactor interceptor

* fix cr

* refactor session.acceptor module:
1. remove WriteDataProcessor: avoid creating unnecessary objects
2. standardize the use of generic types to avoid code warnings

* refactor session.acceptor module:
1. remove WriteDataProcessor: avoid creating unnecessary objects
2. standardize the use of generic types to avoid code warnings

* skip SessionRegistry#process when interceptor execute fail

* Add BufferdDataWritingEngine and simplify DataNodeServiceImpl

* remove Req

* fix code

* fix cr

* SessionRegistryStrategy->ClientRegistrationHook

* remove SyncConfigHandlerStrategy

* move Hook to registry package

* fix testcase

* fix testcase

* refactor session store module

* spring bean ClientRegistrationHook

* fix cr

---------

Co-authored-by: linxin <[email protected]>
Co-authored-by: bjxiaojian <[email protected]>
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