huawei-cloudnative / firmament

The Firmament cluster scheduling platform
http://www.firmament.io
Apache License 2.0
19 stars 9 forks source link

NumNodes in FlowGraphNode is returning wrong value #28

Open shivramsrivastava opened 5 years ago

shivramsrivastava commented 5 years ago

NumNodes in flowgraph is returning wrong value. we should get the size of nodemap as the numNodes , in FlowGraph NumNodes is returning currentid that is returning wrong value.

there is comment about work around, but work around also giving wrong value.

Note :- when adding Node, currentid is increased but not decreasing when deleting the Node/ Note:- nodemap.size() should be used to get exact nodes in the graph.