Skip to content

Simplified conditions, updated variable to final wherever applicable and removed unreachable throws Exception#273

Merged
marsishandsome merged 1 commit intotikv:masterfrom
rahilsh:master
Sep 22, 2021
Merged

Simplified conditions, updated variable to final wherever applicable and removed unreachable throws Exception#273
marsishandsome merged 1 commit intotikv:masterfrom
rahilsh:master

Conversation

@rahilsh
Copy link
Copy Markdown
Contributor

@rahilsh rahilsh commented Sep 11, 2021

What problem does this PR solve?

Simplified conditions, updated variable to final wherever applicable and removed unreachable throws Exception

What is changed and how it works?

Just refactoring and no functional impact

Check List

Tests

  • Unit test

@marsishandsome
Copy link
Copy Markdown
Collaborator

/run-all-tests

@marsishandsome
Copy link
Copy Markdown
Collaborator

please run mvn clean package -DskipTests and the code will be formateed.

@rahilsh
Copy link
Copy Markdown
Contributor Author

rahilsh commented Sep 13, 2021

@marsishandsome Done

@marsishandsome
Copy link
Copy Markdown
Collaborator

/run-all-tests

@marsishandsome
Copy link
Copy Markdown
Collaborator

@rahilsh please add DCO
image

@rahilsh rahilsh closed this Sep 14, 2021
@rahilsh rahilsh reopened this Sep 14, 2021
Signed-off-by: Rahil <rahilrshk@gmail.com>
@rahilsh
Copy link
Copy Markdown
Contributor Author

rahilsh commented Sep 14, 2021

@marsishandsome Done

@rahilsh
Copy link
Copy Markdown
Contributor Author

rahilsh commented Sep 19, 2021

@marsishandsome Please review

@marsishandsome
Copy link
Copy Markdown
Collaborator

/run-all-tests

Copy link
Copy Markdown
Collaborator

@marsishandsome marsishandsome left a comment

Choose a reason for hiding this comment

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

LGTM

@marsishandsome marsishandsome merged commit 37506fe into tikv:master Sep 22, 2021
@rahilsh
Copy link
Copy Markdown
Contributor Author

rahilsh commented Sep 22, 2021

Thanks @marsishandsome

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.

3 participants