Skip to main content

Browse the archive

Revision 4978caaa6f793975f157c13f489ebdb06dc12017 authored by Sagar Vemuri on 28 January 2019, 21:28:31 UTC, committed by Facebook Github Bot on 28 January 2019, 21:33:23 UTC 
Remove a redundant call to TableFileName in CompactionJob::FinishCompactionOutputFile (#4925)
 
Summary:
While stepping through the code I noticed that there is a redundant call to TableFileName.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/4925

Differential Revision: D13845749

Pulled By: sagar0

fbshipit-source-id: 31db45716b4d720e0e0350dd457b49d6f1848e7d
1 parent ee18180
Raw File
file_util.h 
//  Copyright (c) 2011-present, Facebook, Inc.  All rights reserved.
//  This source code is licensed under both the GPLv2 (found in the
//  COPYING file in the root directory) and Apache 2.0 License
//  (found in the LICENSE.Apache file in the root directory).
//
#pragma once
#include <string>

#include "options/db_options.h"
#include "rocksdb/env.h"
#include "rocksdb/status.h"
#include "rocksdb/types.h"

namespace rocksdb {
// use_fsync maps to options.use_fsync, which determines the way that
// the file is synced after copying.
extern Status CopyFile(Env* env, const std::string& source,
                       const std::string& destination, uint64_t size,
                       bool use_fsync);

extern Status CreateFile(Env* env, const std::string& destination,
                         const std::string& contents, bool use_fsync);

extern Status DeleteSSTFile(const ImmutableDBOptions* db_options,
                            const std::string& fname,
                            const std::string& path_to_sync);

}  // namespace rocksdb
back to top