[CBRD-25751] Fix core dump at logging error message from ER_HB_PROCESS_EVENT

[YeunjunLee]

http://jira.cubrid.org/browse/CBRD-25751

• Purpose

  • ER_HB_PROCESS_EVENT는 parameter를 두 개 받는 error message이지만, argument를 하나만 받는 상황이 존재하여 er_estimate_size에서 error 발생

• Implementation

  • Node event에 해당 하는 메시지는 ER_HB_NODE_EVENT로 error message 변경
  • Process event에 해당 하는 메시지는 ER_HB_PROCESS_EVENT로 유지하되, 포맷에 맞도록 변경

[hornetmj]

@YeunjunLee 두 event의 분류 기준을 어떻게 잡으신 건가요?

[YeunjunLee]

@YeunjunLee 두 event의 분류 기준을 어떻게 잡으신 건가요?

다른 message의 경우에는 모두 주어가 (master/slave) node에 대한 것인데, "Server process failure repeated within a short period of time. The current node will be demoted" 는 주어가 process이기 때문에 ER_HB_PROCESS_EVENT라고 판단하였습니다.

하지만, failover/failback message들이 서로 다른 error code로 남는 것이 일관성이 없어 보이기도 합니다. ER_HB_PROCESS_EVENT로 통일 하는 것이 좋을 수도 있을 것 같습니다.

@H2SU @InChiJun 의견 있으시면 주시면 감사하겠습니다.

[InChiJun]

@H2SU @InChiJun 의견 있으시면 주시면 감사하겠습니다.

"Server process failure repeated within a short period of time. The current node will be demoted" 문구는 주어가 process이기 때문에 ER_HB_PROCESS_EVENT를 사용하는 것이 적절해보입니다.
하지만, failover/failback message들은 노드에 대한 에러를 나타내기 때문에 'ER_HB_NODE_EVENT'로 하는 것이 적절해보입니다.

[H2SU]

  else if (hb_Master_host_name[0] != '\0')
{
  snprintf (hb_info_str, HB_INFO_STR_MAX,
	    "%s The master node (%s) has lost its role due to server process problem, such as disk failure",
	    HA_FAILOVER_DIAG_STRING, hb_Master_host_name);
  MASTER_ER_SET (ER_ERROR_SEVERITY, ARG_FILE_LINE, ER_HB_NODE_EVENT, 1, hb_info_str);
}

실제로 위와 같은 에러메세지에서 ER_HB_NODE_EVENT로 분류하여 처리하고 있는 부분이 있는데
연준님이 ER_HB_NODE_EVENT로 분류하신 코드랑 동일한 메세지로 보입니다.

failback/failover messege의 일관성을 위해 ER_HB_PROCESS_EVENT로 변경하여야한다면
동일한 메세지의 에러코드가 출력되고 있는데 하나는 NODE_EVENT이고 다른 하나는 PROCESS_EVENT인 것도
살짝 일관성이 부족해보입니다.

[YeunjunLee]

의미 전달이 잘 안된 것 같습니다. "Server process failure repeated within a short period of time. The current node will be demoted " 또한 failover/failback message의 일부입니다.
위 메시지를 ER_HB_PROCESS_EVENT로 남기고 있는 근거는 message의 내용이 process에 대한 것이기 때문이고,
위 메시지를 ER_HB_NODE_EVENT로 변경하고자 하는 근거는 이 외의 다른 failover/failback message들이 ER_HB_NODE_EVENT로 남겨지고 있기 때문에 일관성을 만들기 위함입니다.

위 message를 살짝 수정하여 node event의 message로 바꾸는 것을 고려중입니다. 아럐는 예시입니다. 피드백 주시면 감사하곘습니다.
The master node failed to restart the server process due to repeated failures within a short period of time. The current node will be demoted.

[hornetmj]

ER_HB_NODE_EVENT와 ER_HB_PROCESS_EVENT를 나누는 원칙(규칙성)이 명확할 필요가 있습니다. 예를들어, hb_resource_job_*()에서는 ER_HB_PROCESS_EVENT를 hb_cluster_job_*()에서는 ER_HB_NODE_EVENT를 적용하는 것입니다. 추가적으로, 같은 콜백 함수에서 ER_HB_PROCESS_EVENT와 ER_HB_NODE_EVENT가 동시에 사용되는 것 역시 추후 메시지 추가시 기준을 잡기 어렵게 만들 수 있습니다.

[YeunjunLee]

해당 message ER_HB_PROCESS_EVENT로 원복하였습니다.