[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16724038#comment-16724038
]
Adam Antal commented on HDFS-7878:
--
We'd like to have this commit in branch-3.0 because of HDFS-14159. I
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16227560#comment-16227560
]
Chris Douglas commented on HDFS-7878:
-
Reverted from 3.0.0, will target 3.1.0.
> API - expose a unique
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16227160#comment-16227160
]
Sergey Shelukhin commented on HDFS-7878:
Thank you for all the work getting this in! I thought it
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16227128#comment-16227128
]
Hudson commented on HDFS-7878:
--
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13168 (See
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16225910#comment-16225910
]
Sean Mackrory commented on HDFS-7878:
-
No other feedback from me - +1 on the bits I understand, like
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16225895#comment-16225895
]
Anu Engineer commented on HDFS-7878:
Thanks for the heads up. Please go ahead.
> API - expose an
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16225867#comment-16225867
]
Chris Douglas commented on HDFS-7878:
-
Thanks [~elgoiri] and [~ste...@apache.org].
If there's no other
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16225724#comment-16225724
]
Steve Loughran commented on HDFS-7878:
--
Bits I Understand are +1 from me
> API - expose an unique
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16225394#comment-16225394
]
Íñigo Goiri commented on HDFS-7878:
---
[~chris.douglas], thanks for pushing this.
V21 LGTM.
For the QA
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16225318#comment-16225318
]
Chris Douglas commented on HDFS-7878:
-
Folks, can this get a definitive review? I think v21 addresses
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16223313#comment-16223313
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16223105#comment-16223105
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=1646#comment-1646
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16220898#comment-16220898
]
Sean Mackrory commented on HDFS-7878:
-
{quote}Wire compatibility with HDFS{quote}
How do commented out
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16219673#comment-16219673
]
Íñigo Goiri commented on HDFS-7878:
---
I'm going through the HDFS code itself and it might be good to
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16219549#comment-16219549
]
Íñigo Goiri commented on HDFS-7878:
---
Thanks [~chris.douglas] for this. I went over the patch and it looks
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16218511#comment-16218511
]
Steve Loughran commented on HDFS-7878:
--
filesystem.md LGTM, +1 for that bit,
Same for the API
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16216140#comment-16216140
]
Chris Douglas commented on HDFS-7878:
-
The javadoc fails to generate a {{\@link}} to a vararg
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16213707#comment-16213707
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16213510#comment-16213510
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16210392#comment-16210392
]
Chris Douglas commented on HDFS-7878:
-
Failure is due to HADOOP-14961
> API - expose an unique file
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16210116#comment-16210116
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16182986#comment-16182986
]
Andrew Wang commented on HDFS-7878:
---
Beta is our compatibility freeze for 3.0.0. I don't know the history
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16182381#comment-16182381
]
Steve Loughran commented on HDFS-7878:
--
I think you can change APIs after a beta...it's only on the
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181721#comment-16181721
]
Sean Mackrory commented on HDFS-7878:
-
{quote}Neither API has been part of a release, yet. {quote}
But
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181641#comment-16181641
]
Chris Douglas commented on HDFS-7878:
-
bq. The following is already annotated as Public and Stable in
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181634#comment-16181634
]
Sean Mackrory commented on HDFS-7878:
-
Looks like this patch is violating some of compatibility
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16172208#comment-16172208
]
Chris Douglas commented on HDFS-7878:
-
Any other feedback on the patch?
As an API for open-exact, one
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16160598#comment-16160598
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16160256#comment-16160256
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16155096#comment-16155096
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16152542#comment-16152542
]
Steve Loughran commented on HDFS-7878:
--
Coming together nicely
h4. RawPathHandle
* L108: any limit
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16151793#comment-16151793
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16142582#comment-16142582
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16141239#comment-16141239
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16140437#comment-16140437
]
Chris Douglas commented on HDFS-7878:
-
bq. There's now a builder API for open/create
There isn't a
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16140397#comment-16140397
]
Andrew Wang commented on HDFS-7878:
---
We still have an outstanding question for Sergey that affects the
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16140300#comment-16140300
]
Chris Douglas commented on HDFS-7878:
-
bq. the "better" calls.
Fine by me. If this JIRA only added an
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16140008#comment-16140008
]
Steve Loughran commented on HDFS-7878:
--
# FS APIs? Pull in [~sanjay.radia]
# There's now a builder API
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16139332#comment-16139332
]
Chris Douglas commented on HDFS-7878:
-
[~sershe] I'm finishing a patch, and will post it this week.
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16138848#comment-16138848
]
Sergey Shelukhin commented on HDFS-7878:
I'd like to attempt to resurrect this. However from
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16100856#comment-16100856
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15963711#comment-15963711
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15515911#comment-15515911
]
Steve Loughran commented on HDFS-7878:
--
I see what you are getting at. No objections from me then.
>
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15515098#comment-15515098
]
Anu Engineer commented on HDFS-7878:
Yes, I think we should use {{BikeShed}} class instead of long,
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15514987#comment-15514987
]
Chris Douglas commented on HDFS-7878:
-
bq. I'd highlight YARN app submission as a key use case; it
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15514658#comment-15514658
]
Anu Engineer commented on HDFS-7878:
[~chris.douglas] +1 on {{BikeShed}} :)
> API - expose an unique
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15514629#comment-15514629
]
Lei (Eddy) Xu commented on HDFS-7878:
-
[~chris.douglas] Thanks a lot for working on it.
I'd prefer to
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15512866#comment-15512866
]
Steve Loughran commented on HDFS-7878:
--
I see: if I do getFileStatus, I get back enough info that I
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15512247#comment-15512247
]
Chris Douglas commented on HDFS-7878:
-
bq. is this somehow going to change end user APIs so a path
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15510055#comment-15510055
]
Steve Loughran commented on HDFS-7878:
--
proposal wise, I'm somewhat confused. is this somehow going to
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15508091#comment-15508091
]
Chris Douglas commented on HDFS-7878:
-
bq. Any changes to FileStatus has to be done so that external
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15507647#comment-15507647
]
Steve Loughran commented on HDFS-7878:
--
hmmm.
* S3a adds an {{S3AFileStatus}} with a new field
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15504298#comment-15504298
]
Chris Douglas commented on HDFS-7878:
-
bq. How about using name service id ("dfs.nameservice.id") here?
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15468143#comment-15468143
]
Jing Zhao commented on HDFS-7878:
-
{{clusterID}} can be a candidate, but it can only be retrieved from
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15456969#comment-15456969
]
Chris Douglas commented on HDFS-7878:
-
[~jingzhao] Good idea on {{FileHandle}}; I prefer that to
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15456737#comment-15456737
]
Jing Zhao commented on HDFS-7878:
-
So the InodeId is actually a file handler? Maybe in that sense to
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15456213#comment-15456213
]
Chris Douglas commented on HDFS-7878:
-
bq. I meant when callers are in different processes/nodes, e.g.
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15456174#comment-15456174
]
Sergey Shelukhin commented on HDFS-7878:
I meant when callers are in different processes/nodes,
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15454081#comment-15454081
]
Hadoop QA commented on HDFS-7878:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15453857#comment-15453857
]
Chris Douglas commented on HDFS-7878:
-
bq. One idea behind open(long/InodeId) is to be able to open
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15453828#comment-15453828
]
Sergey Shelukhin commented on HDFS-7878:
One idea behind open(long/InodeId) is to be able to open
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15453830#comment-15453830
]
Sergey Shelukhin commented on HDFS-7878:
Thanks for picking this up by the way!
> API - expose an
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14492596#comment-14492596
]
Colin Patrick McCabe commented on HDFS-7878:
Symlinks and directories both have
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14490353#comment-14490353
]
Tsz Wo Nicholas Sze commented on HDFS-7878:
---
... I can add null check to get and
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14490354#comment-14490354
]
Tsz Wo Nicholas Sze commented on HDFS-7878:
---
BTW, what happens if we call
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14484342#comment-14484342
]
Hadoop QA commented on HDFS-7878:
-
{color:red}-1 overall{color}. Here are the results of
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14382570#comment-14382570
]
Colin Patrick McCabe commented on HDFS-7878:
I agree with [~jingzhao] that
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14376519#comment-14376519
]
Sergey Shelukhin commented on HDFS-7878:
[~cmccabe] are you ok with the latest
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14370296#comment-14370296
]
Hadoop QA commented on HDFS-7878:
-
{color:red}-1 overall{color}. Here are the results of
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14369753#comment-14369753
]
Jing Zhao commented on HDFS-7878:
-
To me to have a new class extending FileStatus is a
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14369660#comment-14369660
]
Sergey Shelukhin commented on HDFS-7878:
1) Sure
2) That seems error prone; the
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14368360#comment-14368360
]
Colin Patrick McCabe commented on HDFS-7878:
Thanks, [~sershe]. I like version
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14366400#comment-14366400
]
Hadoop QA commented on HDFS-7878:
-
{color:red}-1 overall{color}. Here are the results of
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14364104#comment-14364104
]
Sergey Shelukhin commented on HDFS-7878:
Note: confusingly, HdfsFileStatus is not
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14364389#comment-14364389
]
Colin Patrick McCabe commented on HDFS-7878:
Hi [~jingzhao],
I understand that
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14364067#comment-14364067
]
Jing Zhao commented on HDFS-7878:
-
bq. Colin wrote: if the client makes two different
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14363709#comment-14363709
]
Sergey Shelukhin commented on HDFS-7878:
That is approximately what was done in the
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14361303#comment-14361303
]
Colin Patrick McCabe commented on HDFS-7878:
bq. Jing wrote: Could you please
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14357646#comment-14357646
]
Sergey Shelukhin commented on HDFS-7878:
[~cmccabe] ping?
API - expose an unique
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353579#comment-14353579
]
Sergey Shelukhin commented on HDFS-7878:
ping?
API - expose an unique file
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353616#comment-14353616
]
Jing Zhao commented on HDFS-7878:
-
We also need a full javadoc for the new API. other than
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353776#comment-14353776
]
Jing Zhao commented on HDFS-7878:
-
Thanks for the comments, Colin.
bq. What if the file is
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353755#comment-14353755
]
Colin Patrick McCabe commented on HDFS-7878:
TOCTOU defined here:
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353753#comment-14353753
]
Colin Patrick McCabe commented on HDFS-7878:
Thanks for tackling this!
One big
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353801#comment-14353801
]
Sergey Shelukhin commented on HDFS-7878:
in particular I wonder why getFileStatus
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353762#comment-14353762
]
Hadoop QA commented on HDFS-7878:
-
{color:red}-1 overall{color}. Here are the results of
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353766#comment-14353766
]
Sergey Shelukhin commented on HDFS-7878:
For file opening case, cannot the file be
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14353806#comment-14353806
]
Jing Zhao commented on HDFS-7878:
-
bq. ID allows to overcome this by getting ID first
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14349859#comment-14349859
]
Sergey Shelukhin commented on HDFS-7878:
[~jingzhao] actually getFileId already
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14347653#comment-14347653
]
Sergey Shelukhin commented on HDFS-7878:
[~jingzhao] can you please review?
API -
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14347654#comment-14347654
]
Jing Zhao commented on HDFS-7878:
-
Instead of defining a new DFSFileStatus class, can we
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14347938#comment-14347938
]
Hadoop QA commented on HDFS-7878:
-
{color:red}-1 overall{color}. Here are the results of
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14347850#comment-14347850
]
Hadoop QA commented on HDFS-7878:
-
{color:red}-1 overall{color}. Here are the results of
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14347889#comment-14347889
]
Sergey Shelukhin commented on HDFS-7878:
This QA was for old patch...
API -
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14345880#comment-14345880
]
Jing Zhao commented on HDFS-7878:
-
bq. dfs.getClient().getFileInfo(path)
yes, then you can
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14345824#comment-14345824
]
Sergey Shelukhin commented on HDFS-7878:
HdfsFileStatus is not inherited from
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14345775#comment-14345775
]
Suresh Srinivas commented on HDFS-7878:
---
[~sershe], FileSystem#getFileStatus returns
[
https://issues.apache.org/jira/browse/HDFS-7878?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14345930#comment-14345930
]
Sergey Shelukhin commented on HDFS-7878:
Can you make it public? Or better yet add
99 matches
Mail list logo