Skip to content
  • Mayank Agarwal's avatar
    Fix build caused by DeleteFile not tolerating / at the beginning · ab5c5c28
    Mayank Agarwal authored
    Summary: db->DeleteFile calls ParseFileName to check name that was returned for sst file. Now, sst filename is returned using TableFileName which uses MakeFileName. This puts a / at the front of the name and ParseFileName doesn't like that. Changed ParseFileName to tolerate /s at the beginning. The test delet_file_test used to pass earlier because this behaviour of MakeFileName had been changed a while back to not return a / during which delete_file_test was checked in. But MakeFileName had to be reverted to add / at the front because GetLiveFiles used at many places outside rocksdb used the previous behaviour of MakeFileName.
    
    Test Plan: make;./delete_filetest;make all check
    
    Reviewers: dhruba, haobo, vamsi
    
    Reviewed By: dhruba
    
    CC: leveldb
    
    Differential Revision: https://reviews.facebook.net/D12663
    ab5c5c28