Skip to content

Commit f72ab92

Browse files
authored
Handle DataNode failures in replicated writes (#164)
1 parent c7d40bd commit f72ab92

File tree

8 files changed

+599
-302
lines changed

8 files changed

+599
-302
lines changed

Cargo.lock

+2-2
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

rust/minidfs/src/main/java/main/Main.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ public static void main(String args[]) throws Exception {
106106
} else {
107107
MiniDFSNNTopology nnTopology = generateTopology(flags, hdfsConf);
108108

109-
int numDataNodes = 1;
109+
int numDataNodes = 3;
110110
if (flags.contains("ec")) {
111111
// Enough for the largest EC policy
112112
numDataNodes = 14;

rust/src/file.rs

+2-6
Original file line numberDiff line numberDiff line change
@@ -208,9 +208,7 @@ impl FileWriter {
208208
// Not appending to an existing block, just create a new one
209209
// If there's an existing block writer, close it first
210210
let extended_block = if let Some(block_writer) = self.block_writer.take() {
211-
let extended_block = block_writer.get_extended_block();
212-
block_writer.close().await?;
213-
Some(extended_block)
211+
Some(block_writer.close().await?)
214212
} else {
215213
None
216214
};
@@ -265,9 +263,7 @@ impl FileWriter {
265263
pub async fn close(&mut self) -> Result<()> {
266264
if !self.closed {
267265
let extended_block = if let Some(block_writer) = self.block_writer.take() {
268-
let extended_block = block_writer.get_extended_block();
269-
block_writer.close().await?;
270-
Some(extended_block)
266+
Some(block_writer.close().await?)
271267
} else {
272268
None
273269
};

0 commit comments

Comments
 (0)