apache / incubator-pegasus

Apache Pegasus - A horizontally scalable, strongly consistent and high-performance key-value store
https://pegasus.apache.org/
Apache License 2.0
1.99k stars 312 forks source link

Wrong logging format was found in meta server_state #1157

Closed empiredan closed 2 years ago

empiredan commented 2 years ago

Some logs are found for meta server as follows:

F2022-09-09 17:02:17.184 (1662714137184230445 11231)   meta.meta_state0.0202000700000001: server_state.cpp:895:on_config_sync(): assertion expression: false
F2022-09-09 17:02:17.184 (1662714137184247617 11231)   meta.meta_state0.0202000700000001: server_state.cpp:895:on_config_sync(): gpid({}) on node({}) is not exist on meta server, administrator should check consistency of meta data

The key point is gpid({}) on node({}) in logs of meta server rather than the fatal level, which is the result of connecting to a wrong replica server. gpid({}) on node({}) must be caused by invalid logging format.

empiredan commented 2 years ago

This issue has been fixed by https://github.com/apache/incubator-pegasus/pull/1217.