[GitHub] [hadoop] bgaborg commented on issue #1302: HADOOP-16138. hadoop fs mkdir / of nonexistent abfs container raises NPE

[GitBox]

bgaborg commented on issue #1302: HADOOP-16138. hadoop fs mkdir / of 
nonexistent abfs container raises NPE
URL: https://github.com/apache/hadoop/pull/1302#issuecomment-533181106
 
 
   > looking at the mkdir() code, i can see that it raises a PathIOE on failure 
without any text. Do you think we should add a cause like "mkdir failed"?
   
   yeah, I can add since I'm already there.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

[GitHub] [hadoop] bgaborg commented on issue #1302: HADOOP-16138. hadoop fs mkdir / of nonexistent abfs container raises NPE

[GitBox]

bgaborg commented on issue #1302: HADOOP-16138. hadoop fs mkdir / of 
nonexistent abfs container raises NPE
URL: https://github.com/apache/hadoop/pull/1302#issuecomment-529234167
 
 
   I haven't figured out yet where to handle that. We can talk about that
   offline - it will be just a few minutes.
   
   On Fri, 6 Sep 2019 at 22:34, Steve Loughran 
   wrote:
   
   > OK, so this is the test. What about the underlying NPE?
   >
   > —
   > You are receiving this because you authored the thread.
   > Reply to this email directly, view it on GitHub
   > 
,
   > or mute the thread
   > 

   > .
   >
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

[GitHub] [hadoop] bgaborg commented on issue #1302: HADOOP-16138. hadoop fs mkdir / of nonexistent abfs container raises NPE

[GitBox]

bgaborg commented on issue #1302: HADOOP-16138. hadoop fs mkdir / of 
nonexistent abfs container raises NPE
URL: https://github.com/apache/hadoop/pull/1302#issuecomment-523951084
 
 
   @mackrorysd I found out what's happening. We auto-create all containers 
during the test run, so I should turn that feature off for this test. I forgot 
to do that, and so I ended up a lot of random containers in the testing 
account. 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

[GitHub] [hadoop] bgaborg commented on issue #1302: HADOOP-16138. hadoop fs mkdir / of nonexistent abfs container raises NPE

[GitBox]

bgaborg commented on issue #1302: HADOOP-16138. hadoop fs mkdir / of 
nonexistent abfs container raises NPE
URL: https://github.com/apache/hadoop/pull/1302#issuecomment-521974548
 
 
   Yeah, I need to add a message which can be understood easily.
   
   The hard thing is that this message comes from 
`org.apache.hadoop.fs.shell.Mkdir#processPath`:
   ```java
 @Override
 protected void processPath(PathData item) throws IOException {
   if (item.stat.isDirectory()) {
 if (!createParents) {
   throw new PathExistsException(item.toString());
 }
   } else {
 throw new PathIsNotDirectoryException(item.toString());
   }
 }
   ```
   
   so not from the `org.apache.hadoop.fs.azurebfs.AzureBlobFileSystem`, so it's 
a little more tricky than I expected (at least for me) to solve this nicely.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org