Chakra ETNode Comm Size Read Error

[zzudongxiang]

Describe the Bug

When calling the et_feeder_node->comm_size() method of ETFeederNode, it is found that the read comm size is 0, but the expected result should not be 0.

After investigation, the acquisition failed because the type obtained in src/feeder/et_feeder_node.cpp:28 was attr.int64_val() instead of attr.uint64_val(). it causes the method to return a result of 0

// in src/feeder/et_feeder_node.cpp:28
ETFeederNode::ETFeederNode(std::shared_ptr<ChakraProtoMsg::Node> node) {
...
    } else if (attr_name == "comm_size") {
      this->comm_size_ = attr.int64_val(); // attr.uint64_val() method should be used here
...
}

Looking forward to fixing this bug and checking if there are similar issues elsewhere.

[srinivas212]

@zzudongxiang thanks for reporting the issue. Can you file a PR?