Skip to content
This repository was archived by the owner on Jun 20, 2025. It is now read-only.

Remove unused variables in fboss/agent/state/Interface.cpp #2395

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Remove unused variables in fboss/agent/state/Interface.cpp #2395

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 0 additions & 9 deletions fbpcs/data_processing/sharding/GenericSharder.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,15 +65,6 @@ void strReplaceNullColumnWithEmpty(std::string& str) {
static const std::string kIdColumnPrefix = "id_";
static const std::string numIds = "num_ids";

/*
The chunk size for writing to cloud storage (currently
only AWS S3) must be greater than 5 MB, per the AWS
documentation. Otherwise multipart upload will fail.

The number below is 5 MB in bytes.
*/
static const uint64_t kBufferedWriterChunkSize = 5'242'880;

std::vector<std::string> GenericSharder::genOutputPaths(
const std::string& outputBasePath,
std::size_t startIndex,
Expand Down