Skip to content

Conversation

@JackieTien97
Copy link
Contributor

No description provided.

@JackieTien97 JackieTien97 requested a review from qiaojialin June 8, 2020 12:58
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 8, 2020

SonarCloud Quality Gate failed.

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

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (1.8.0_242) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@samperson1997
Copy link
Contributor

Hi, should this PR be merged into rel/0.10 ?

@jixuan1989
Copy link
Member

Hi, should this PR be merged into rel/0.10 ?

Ok.

By the way, pay attention to the code smell (use log.isWarnEnable() before calling log.warn() because there are too many string concat.)

@qiaojialin qiaojialin merged commit 92f014f into master Jun 9, 2020
@qiaojialin qiaojialin deleted the TyDeleteBug branch June 9, 2020 01:20
Copilot AI added a commit to OneSizeFitsQuorum/iotdb that referenced this pull request Jan 7, 2026
HTHou pushed a commit that referenced this pull request Jan 8, 2026
…ility-check workflow (#16995)

* Initial plan

* Apply improvements from Ratis PR #1328 to vulnerability-check workflow

Co-authored-by: OneSizeFitsQuorum <[email protected]>

* Add explicit permissions block for security best practices

Co-authored-by: OneSizeFitsQuorum <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
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