Skip to content

Minor code cleanup#13113

Merged
phd3 merged 1 commit intotrinodb:masterfrom
tangjiangling:minor-code-cleanup
Jul 7, 2022
Merged

Minor code cleanup#13113
phd3 merged 1 commit intotrinodb:masterfrom
tangjiangling:minor-code-cleanup

Conversation

@tangjiangling
Copy link
Copy Markdown
Member

@tangjiangling tangjiangling commented Jul 7, 2022

Extracted 1st commit from #13042.

  • Fix typo
  • Inline variable
  • Remove unused parameter

Description

Is this change a fix, improvement, new feature, refactoring, or other?

Code cleanup.

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

N/A.

How would you describe this change to a non-technical end user or system administrator?

N/A.

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.

Release notes

(x) No release notes entries required.

@cla-bot cla-bot bot added the cla-signed label Jul 7, 2022
@tangjiangling tangjiangling marked this pull request as ready for review July 7, 2022 06:14
@tangjiangling tangjiangling requested a review from arhimondr July 7, 2022 06:14
@tangjiangling tangjiangling added the no-release-notes This pull request does not require release notes entry label Jul 7, 2022
@tangjiangling
Copy link
Copy Markdown
Member Author

( Fix ci / maven-checks (11) )

@arhimondr
Copy link
Copy Markdown
Contributor

Please change the commit message according to the guidelines. For example: Remove unused code and fix typos in task execution tests

This commit includes:

- Fix typo
- Inline variable
- Remove unused parameter
@tangjiangling
Copy link
Copy Markdown
Member Author

AC

@phd3 phd3 merged commit 432d289 into trinodb:master Jul 7, 2022
@github-actions github-actions bot added this to the 390 milestone Jul 7, 2022
@tangjiangling tangjiangling deleted the minor-code-cleanup branch July 8, 2022 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

3 participants