Github user zcorrea commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1457#discussion_r171408008
--- Diff: core/sqf/monitor/linux/cluster.h ---
@@ -229,6 +235,7 @@ class CCluster
int configPNodesMax_; // max # of physical nodes that can be
configured
int *NodeMap; // Mapping of Node ranks to COMM_WORLD ranks
int TmLeaderNid; // Nid of currently assigned TM Leader node
+ int MonitorLeaderPNid; // PNid of currently assigned Monitor
leader node
--- End diff --
Yes, we do. Noted these anomalies in the code this morning and will change
this to the followed convention. Thanks for pointing this out.
---
