Skip to content

[Improvement][common] Add UnsupportedOperationException for utils construct#3381

Merged
lgcareer merged 2 commits intoapache:devfrom
yangyichao-mango:feature-UnsupportedOperationException
Sep 2, 2020
Merged

[Improvement][common] Add UnsupportedOperationException for utils construct#3381
lgcareer merged 2 commits intoapache:devfrom
yangyichao-mango:feature-UnsupportedOperationException

Conversation

@yangyichao-mango
Copy link
Copy Markdown
Contributor

What is the purpose of the pull request

Add UnsupportedOperationException for utils construct to avoid the construct of utils and improvement the robust of code

Brief change log

  • All utils in common module, add the UnsupportedOperationException in every utils construct

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Aug 2, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Copy Markdown
Contributor

@qiaozhanwei qiaozhanwei left a comment

Choose a reason for hiding this comment

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

please resolve conflicts
+1

@CalvinKirs
Copy link
Copy Markdown
Member

please resolve conflicts

@yangyichao-mango yangyichao-mango force-pushed the feature-UnsupportedOperationException branch from 7d078dd to 5d16cff Compare September 1, 2020 14:12
@yangyichao-mango yangyichao-mango force-pushed the feature-UnsupportedOperationException branch from 5d16cff to cfc9a51 Compare September 1, 2020 14:13
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Sep 1, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 1 Code Smell

40.0% 40.0% Coverage
0.4% 0.4% Duplication

@yangyichao-mango
Copy link
Copy Markdown
Contributor Author

@CalvinKirs @qiaozhanwei done.

Copy link
Copy Markdown
Member

@CalvinKirs CalvinKirs left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Copy Markdown
Contributor

@lgcareer lgcareer left a comment

Choose a reason for hiding this comment

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

+1

@lgcareer lgcareer merged commit ac4ed94 into apache:dev Sep 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants