Oleg Danilov created HDFS-12315:
-----------------------------------

             Summary: Use Path instead of String in the 
TestHdfsAdmin.verifyOpenFiles()
                 Key: HDFS-12315
                 URL: https://issues.apache.org/jira/browse/HDFS-12315
             Project: Hadoop HDFS
          Issue Type: Bug
            Reporter: Oleg Danilov
            Priority: Trivial


closedFiles is a set of Path, therefor closedFiles.contains(String) doesn't 
make sense.

lines 252-261:
{code:java}
  private void verifyOpenFiles(HashSet<Path> closedFiles,
      HashMap<Path, FSDataOutputStream> openFileMap) throws IOException {
    HdfsAdmin hdfsAdmin = new HdfsAdmin(FileSystem.getDefaultUri(conf), conf);
    HashSet<Path> openFiles = new HashSet<>(openFileMap.keySet());
    RemoteIterator<OpenFileEntry> openFilesRemoteItr =
        hdfsAdmin.listOpenFiles();
    while (openFilesRemoteItr.hasNext()) {
      String filePath = openFilesRemoteItr.next().getFilePath();
      assertFalse(filePath + " should not be listed under open files!",
          closedFiles.contains(filePath));
{code}




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org

Reply via email to