Skip to content

[ISSUE #3862] Change NullPointException to be explicit#3863

Merged
yu199195 merged 1 commit intoapache:masterfrom
MiracleDx:feature/client_config
Aug 23, 2022
Merged

[ISSUE #3862] Change NullPointException to be explicit#3863
yu199195 merged 1 commit intoapache:masterfrom
MiracleDx:feature/client_config

Conversation

@MiracleDx
Copy link
Copy Markdown
Contributor

@MiracleDx MiracleDx commented Aug 19, 2022

About #3862

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 19, 2022

Codecov Report

Merging #3863 (fdd4bc2) into master (7909785) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3863      +/-   ##
============================================
- Coverage     64.76%   64.74%   -0.02%     
+ Complexity     6099     6097       -2     
============================================
  Files           898      898              
  Lines         24729    24729              
  Branches       2240     2240              
============================================
- Hits          16015    16011       -4     
- Misses         7307     7311       +4     
  Partials       1407     1407              
Impacted Files Coverage Δ
...controller/ShenyuClientHttpRegistryController.java 77.77% <0.00%> (-22.23%) ⬇️
...ruptor/RegisterClientServerDisruptorPublisher.java 52.94% <0.00%> (-11.77%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@yu199195 yu199195 merged commit 4b4adfd into apache:master Aug 23, 2022
@dengliming dengliming linked an issue Aug 27, 2022 that may be closed by this pull request
1 task
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.

[BUG] When client properties is null or invalid while throw NullPointException

3 participants