Skip to content

LOGGER.error(e.getMessage()) IS REALLY A BAD IDEA#14137

Merged
OneSizeFitsQuorum merged 3 commits intoapache:masterfrom
liyuheng55555:Fix/fxxk-e.getMessage()
Nov 21, 2024
Merged

LOGGER.error(e.getMessage()) IS REALLY A BAD IDEA#14137
OneSizeFitsQuorum merged 3 commits intoapache:masterfrom
liyuheng55555:Fix/fxxk-e.getMessage()

Conversation

@liyuheng55555
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

@OneSizeFitsQuorum OneSizeFitsQuorum left a comment

Choose a reason for hiding this comment

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

LGTM

@OneSizeFitsQuorum OneSizeFitsQuorum merged commit ec5d2ca into apache:master Nov 21, 2024
HxpSerein pushed a commit to HxpSerein/iotdb that referenced this pull request Dec 9, 2024
OneSizeFitsQuorum pushed a commit that referenced this pull request Dec 9, 2024
…ev 1.3 (#14361)

* Delete system.properties in a better way #14080

(cherry picked from commit 6197bd0)

* remove procedure no longer acquire node lock (#14115)

(cherry picked from commit 6242d93)

* LOGGER.error(e.getMessage()) IS REALLY A BAD IDEA (#14137)

(cherry picked from commit ec5d2ca)

---------

Co-authored-by: Li Yu Heng <liyuheng55555@126.com>
Co-authored-by: Peng Junzhi <78788603+Pengzna@users.noreply.github.com>
Caideyipi pushed a commit to Caideyipi/iotdb that referenced this pull request Mar 25, 2026
…ev 1.3 (apache#14361)

* Delete system.properties in a better way apache#14080

(cherry picked from commit 6197bd0)

* remove procedure no longer acquire node lock (apache#14115)

(cherry picked from commit 6242d93)

* LOGGER.error(e.getMessage()) IS REALLY A BAD IDEA (apache#14137)

(cherry picked from commit ec5d2ca)

---------

Co-authored-by: Li Yu Heng <liyuheng55555@126.com>
Co-authored-by: Peng Junzhi <78788603+Pengzna@users.noreply.github.com>
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.

2 participants