[ 
https://issues.apache.org/jira/browse/HDFS-17068?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17741236#comment-17741236
 ] 

ASF GitHub Bot commented on HDFS-17068:
---------------------------------------

xinglin commented on code in PR #5809:
URL: https://github.com/apache/hadoop/pull/5809#discussion_r1256899902


##########
hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/datanode/extdataset/ExternalDatasetImpl.java:
##########
@@ -477,4 +482,7 @@ public MountVolumeMap getMountVolumeMap() {
   public List<FsVolumeImpl> getVolumeList() {
     return null;
   }
+
+  @Override
+  public void setLastDirScannerFinishTime(long time) {}

Review Comment:
   Is there a reason that in one test case, we throw an unsupported exception 
while in this test class, it is a no-op? can we make both behave the same way? 





> Datanode should record last directory scan time.
> ------------------------------------------------
>
>                 Key: HDFS-17068
>                 URL: https://issues.apache.org/jira/browse/HDFS-17068
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: farmmamba
>            Assignee: farmmamba
>            Priority: Minor
>              Labels: pull-request-available
>
> I think it is useful for us to record last directory scan time for one 
> datanode. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to