This is an automated email from the ASF dual-hosted git repository.
dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 540bce4d1b3 [typo](log) Let env lock msg more distinct (#27493)
540bce4d1b3 is described below
commit 540bce4d1b32263e78f18e77dbaa071374c778f2
Author: Lei Zhang <[email protected]>
AuthorDate: Thu Nov 23 23:03:06 2023 +0800
[typo](log) Let env lock msg more distinct (#27493)
---
fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java | 10 +++++-----
.../java/org/apache/doris/datasource/InternalCatalog.java | 14 ++++++--------
2 files changed, 11 insertions(+), 13 deletions(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java
b/fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java
index 941e24599b7..78f83bb7c5d 100755
--- a/fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java
@@ -855,7 +855,7 @@ public class Env {
// to see which thread held this lock for long time.
Thread owner = lock.getOwner();
if (owner != null) {
- LOG.info("catalog lock is held by: {}",
Util.dumpThread(owner, 10));
+ LOG.info("env lock is held by: {}",
Util.dumpThread(owner, 10));
}
if (mustLock) {
@@ -866,7 +866,7 @@ public class Env {
}
return true;
} catch (InterruptedException e) {
- LOG.warn("got exception while getting catalog lock", e);
+ LOG.warn("got exception while getting env lock", e);
if (mustLock) {
continue;
} else {
@@ -2634,7 +2634,7 @@ public class Env {
public void addFrontend(FrontendNodeType role, String host, int
editLogPort) throws DdlException {
if (!tryLock(false)) {
- throw new DdlException("Failed to acquire catalog lock. Try
again");
+ throw new DdlException("Failed to acquire env lock. Try again");
}
try {
Frontend fe = checkFeExist(host, editLogPort);
@@ -2680,7 +2680,7 @@ public class Env {
public void modifyFrontendHost(String nodeName, String destHost) throws
DdlException {
if (!tryLock(false)) {
- throw new DdlException("Failed to acquire catalog lock. Try
again");
+ throw new DdlException("Failed to acquire env lock. Try again");
}
try {
Frontend fe = getFeByName(nodeName);
@@ -2711,7 +2711,7 @@ public class Env {
throw new DdlException("can not drop current master node.");
}
if (!tryLock(false)) {
- throw new DdlException("Failed to acquire catalog lock. Try
again");
+ throw new DdlException("Failed to acquire env lock. Try again");
}
try {
Frontend fe = checkFeExist(host, port);
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/datasource/InternalCatalog.java
b/fe/fe-core/src/main/java/org/apache/doris/datasource/InternalCatalog.java
index aa8d40afc23..a8f548185c9 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/datasource/InternalCatalog.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/datasource/InternalCatalog.java
@@ -330,14 +330,12 @@ public class InternalCatalog implements
CatalogIf<Database> {
while (true) {
try {
if (!lock.tryLock(Config.catalog_try_lock_timeout_ms,
TimeUnit.MILLISECONDS)) {
- if (LOG.isDebugEnabled()) {
- // to see which thread held this lock for long time.
- Thread owner = lock.getOwner();
- if (owner != null) {
- // There are many catalog timeout during
regression test
- // And this timeout should not happen very often,
so it could be info log
- LOG.info("catalog lock is held by: {}",
Util.dumpThread(owner, 10));
- }
+ // to see which thread held this lock for long time.
+ Thread owner = lock.getOwner();
+ if (owner != null) {
+ // There are many catalog timeout during regression
test
+ // And this timeout should not happen very often, so
it could be info log
+ LOG.info("catalog lock is held by: {}",
Util.dumpThread(owner, 10));
}
if (mustLock) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
