From 253bc91953a0accb4f1a1643543b4676e59445d8 Mon Sep 17 00:00:00 2001 From: yaphet <664737951@qq.com> Date: Thu, 15 Jun 2023 16:55:38 -0700 Subject: [PATCH] Move the status judgment into the block (#11534) Summary: Pull Request resolved: https://github.com/facebook/rocksdb/pull/11534 Reviewed By: cbi42 Differential Revision: D46732248 Pulled By: ajkr fbshipit-source-id: c9360866c35a2c436ab83b85a14ed7bd43a88d95 --- db/external_sst_file_ingestion_job.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/db/external_sst_file_ingestion_job.cc b/db/external_sst_file_ingestion_job.cc index f8716e5f4..2f8c9779a 100644 --- a/db/external_sst_file_ingestion_job.cc +++ b/db/external_sst_file_ingestion_job.cc @@ -708,9 +708,9 @@ Status ExternalSstFileIngestionJob::GetIngestedFileInfo( ro.readahead_size = ingestion_options_.verify_checksums_readahead_size; status = table_reader->VerifyChecksum( ro, TableReaderCaller::kExternalSSTIngestion); - } - if (!status.ok()) { - return status; + if (!status.ok()) { + return status; + } } // Get the external file properties