Skip to content

Commit 29595ff

Browse files
committed
fix:optimize logging.
1 parent 6d20ef1 commit 29595ff

File tree

3 files changed

+8
-4
lines changed

3 files changed

+8
-4
lines changed

polaris-common/polaris-logging/src/main/resources/polaris-log4j2.xml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@
22

33
<Configuration status="WARN">
44
<Appenders>
5+
<Console name="Console" target="SYSTEM_OUT">
6+
<PatternLayout pattern="%d{HH:mm:ss.SSS} [%t] %-5level %logger{36} - %msg%n"/>
7+
</Console>
58
<RollingFile name="POLARIS_LOG_FILE" fileName="${sys:polaris.log.home}/polaris.log"
69
filePattern="${sys:polaris.log.home}/polaris.log.%d{yyyy-MM-dd}.%i">
710
<PatternLayout>
@@ -65,6 +68,9 @@
6568
</Appenders>
6669

6770
<Loggers>
71+
<Root level="error">
72+
<AppenderRef ref="Console"/>
73+
</Root>
6874
<Logger name="com.tencent.polaris"
6975
level="${sys:polaris.log.level:-info}"
7076
additivity="false">

polaris-plugins/polaris-plugins-connector/connector-polaris-grpc/src/main/java/com/tencent/polaris/plugins/connector/grpc/GrpcConnector.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -257,9 +257,7 @@ public void onNext(DiscoverResponse value) {
257257
@Override
258258
public void onError(Throwable t) {
259259
countDownLatch.countDown();
260-
LOG.error("[ServerConnector] fail to acquire check event type {}", eventType, t);
261-
throw new PolarisException(ErrorCode.NETWORK_ERROR,
262-
"[ServerConnector] fail to acquire check event type " + eventType, t);
260+
LOG.warn("[ServerConnector] fail to acquire check event type {}, cause: {}", eventType, t.getMessage());
263261
}
264262

265263
@Override

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464

6565
<properties>
6666
<!-- Project revision -->
67-
<revision>1.12.6</revision>
67+
<revision>1.12.7</revision>
6868

6969
<timestamp>${maven.build.timestamp}</timestamp>
7070
<skip.maven.deploy>false</skip.maven.deploy>

0 commit comments

Comments
 (0)