Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1457#discussion_r171320342
  
    --- 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 --
    
    In much of the SQL code, the convention is that member names end in an 
underscore. Do we follow that convention in the monitor?


---

Reply via email to