It will more useful for debugging purpose where user can see how many blocks got schduled for DN
Uploaded the patch. Kindly Review.
This message was automatically generated.
Patch generated 1 ASF License warnings.
not related to this patch..This is introduced in HDFS-9005.
can anybody review this..?
Ravi Prakash can you review this..?
What is the motivation behind this Brahma? Besides I see this returns (currApproxBlocksScheduled.sum() + prevApproxBlocksScheduled.sum() . If its to check how busy a datanode is at present, a lot factors come into play. e.g. the block sizes of the blocks, the heartbeat frequency, other readers, writers etc.
If its to check how busy a datanode is at present, a lot factors come into play. e.g. the block sizes of the blocks, the heartbeat frequency, other readers, writers etc.
Mainly for debugging purpose..It will be visible to the admins which DN's got scheduled more blocks...Block size can be checked, heabeat frequncy also checked through last contact.
How about making it scheduled count as a "tooltip" for the "blocks" column, similar to 'old jsp webui' so that extra column need not be added.?
I am ok with making tooltip.. uploaded the patch..Kindly review..
Tooltip patch looks good. +1
Ravi Prakash, Can you also take a look?
I am planning to commit this tomorrow, unless there is something to be addressed, from Ravi.
I'm fine with exposing the information as a tooltip.
Committed to trunk, branch-2 and branch-2.8.
Thanks Brahma Reddy Battula for the contribution.
Thanks Ravi Prakash for reviews.
SUCCESS: Integrated in Hadoop-trunk-Commit #9855 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9855/)
HDFS-10217. show 'blockScheduled' tooltip in datanodes table. (vinayakumarb: rev 2dab64da2ff50bbb7937aad2d376eacd4ed320aa)
Vinayakumar B thanks for review and commit and thanks to Ravi Prakash for additional review..