[jira] [Updated] (HADOOP-14505) simplify mkdirs() after S3Guard delete tracking change
[ https://issues.apache.org/jira/browse/HADOOP-14505?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Steve Loughran updated HADOOP-14505: Resolution: Fixed Fix Version/s: HADOOP-13345 Status: Resolved (was: Patch Available) +1 reran all tests, apart from the ongoing SSE-C test failure, all is well. Committed & pushed up Thanks > simplify mkdirs() after S3Guard delete tracking change > -- > > Key: HADOOP-14505 > URL: https://issues.apache.org/jira/browse/HADOOP-14505 > Project: Hadoop Common > Issue Type: Sub-task > Components: fs/s3 >Reporter: Aaron Fabbri >Assignee: Sean Mackrory >Priority: Minor > Fix For: HADOOP-13345 > > Attachments: HADOOP-14505-HADOOP-13345.001.patch > > > I noticed after reviewing the S3Guard delete tracking changes for > HADOOP-13760, that mkdirs() can probably be simplified, replacing the use of > checkPathForDirectory() with a simple getFileStatus(). > Creating a separate JIRA so these changes can be reviewed / tested in > isolation. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Updated] (HADOOP-14505) simplify mkdirs() after S3Guard delete tracking change
[ https://issues.apache.org/jira/browse/HADOOP-14505?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sean Mackrory updated HADOOP-14505: --- Assignee: Sean Mackrory Status: Patch Available (was: Open) Yes it is! That patch still applies cleanly and I just reran unit tests with every valid combo of -Ds3guard, -Ddynamo and -Dauth. I had an issue with SSEC tests and a transient "Bad Request" on testRecursiveRootListing. Neither is related - I suspect the other SSEC test just needs to be skipped on S3Guard like we do with the others... > simplify mkdirs() after S3Guard delete tracking change > -- > > Key: HADOOP-14505 > URL: https://issues.apache.org/jira/browse/HADOOP-14505 > Project: Hadoop Common > Issue Type: Sub-task > Components: fs/s3 >Reporter: Aaron Fabbri >Assignee: Sean Mackrory >Priority: Minor > Attachments: HADOOP-14505-HADOOP-13345.001.patch > > > I noticed after reviewing the S3Guard delete tracking changes for > HADOOP-13760, that mkdirs() can probably be simplified, replacing the use of > checkPathForDirectory() with a simple getFileStatus(). > Creating a separate JIRA so these changes can be reviewed / tested in > isolation. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Updated] (HADOOP-14505) simplify mkdirs() after S3Guard delete tracking change
[ https://issues.apache.org/jira/browse/HADOOP-14505?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sean Mackrory updated HADOOP-14505: --- Attachment: HADOOP-14505-HADOOP-13345.001.patch Attaching a patch. Is mostly changes originally in HADOOP-14457 patches, but a few changes to rebase on top of later work. Still actually depends on HADOOP-14457 to go in first because there is some overlap to the changes. > simplify mkdirs() after S3Guard delete tracking change > -- > > Key: HADOOP-14505 > URL: https://issues.apache.org/jira/browse/HADOOP-14505 > Project: Hadoop Common > Issue Type: Sub-task > Components: fs/s3 >Reporter: Aaron Fabbri >Priority: Minor > Attachments: HADOOP-14505-HADOOP-13345.001.patch > > > I noticed after reviewing the S3Guard delete tracking changes for > HADOOP-13760, that mkdirs() can probably be simplified, replacing the use of > checkPathForDirectory() with a simple getFileStatus(). > Creating a separate JIRA so these changes can be reviewed / tested in > isolation. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org