[jira] [Commented] (HBASE-7439) HFileLink should not use the configuration from the Filesystem

2012-12-28 Thread Jonathan Hsieh (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13540507#comment-13540507
 ] 

Jonathan Hsieh commented on HBASE-7439:
---

currently testing before committing 

> HFileLink should not use the configuration from the Filesystem
> --
>
> Key: HBASE-7439
> URL: https://issues.apache.org/jira/browse/HBASE-7439
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client, master, regionserver, snapshots, Zookeeper
>Affects Versions: hbase-6055
>Reporter: Matteo Bertozzi
>Assignee: Matteo Bertozzi
>Priority: Blocker
> Fix For: hbase-6055, 0.96.0
>
> Attachments: HBASE-7439-v0.patch
>
>
> This is related to HBASE-7423 and HBASE-7422
> since the fs.getConf() can be the unexpected one, we should avoid to use that 
> to get the root dir

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7439) HFileLink should not use the configuration from the Filesystem

2012-12-26 Thread Ted Yu (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13539783#comment-13539783
 ] 

Ted Yu commented on HBASE-7439:
---

+1 on patch.

> HFileLink should not use the configuration from the Filesystem
> --
>
> Key: HBASE-7439
> URL: https://issues.apache.org/jira/browse/HBASE-7439
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client, master, regionserver, snapshots, Zookeeper
>Affects Versions: hbase-6055
>Reporter: Matteo Bertozzi
>Assignee: Matteo Bertozzi
>Priority: Blocker
> Fix For: hbase-6055, 0.96.0
>
> Attachments: HBASE-7439-v0.patch
>
>
> This is related to HBASE-7423 and HBASE-7422
> since the fs.getConf() can be the unexpected one, we should avoid to use that 
> to get the root dir

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7439) HFileLink should not use the configuration from the Filesystem

2012-12-26 Thread Himanshu Vashishtha (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13539770#comment-13539770
 ] 

Himanshu Vashishtha commented on HBASE-7439:


+1

Yes, we should get rid of the method as its not used; it looks more like a util 
method too.

> HFileLink should not use the configuration from the Filesystem
> --
>
> Key: HBASE-7439
> URL: https://issues.apache.org/jira/browse/HBASE-7439
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client, master, regionserver, snapshots, Zookeeper
>Affects Versions: hbase-6055
>Reporter: Matteo Bertozzi
>Assignee: Matteo Bertozzi
>Priority: Blocker
> Fix For: hbase-6055, 0.96.0
>
> Attachments: HBASE-7439-v0.patch
>
>
> This is related to HBASE-7423 and HBASE-7422
> since the fs.getConf() can be the unexpected one, we should avoid to use that 
> to get the root dir

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7439) HFileLink should not use the configuration from the Filesystem

2012-12-26 Thread Matteo Bertozzi (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13539734#comment-13539734
 ] 

Matteo Bertozzi commented on HBASE-7439:


internal and not used... I can add to it a conf argument if you want.
HRegion uses StoreFile.getHDFSBlockDistribution()

> HFileLink should not use the configuration from the Filesystem
> --
>
> Key: HBASE-7439
> URL: https://issues.apache.org/jira/browse/HBASE-7439
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client, master, regionserver, snapshots, Zookeeper
>Affects Versions: hbase-6055
>Reporter: Matteo Bertozzi
>Assignee: Matteo Bertozzi
>Priority: Blocker
> Fix For: hbase-6055, 0.96.0
>
> Attachments: HBASE-7439-v0.patch
>
>
> This is related to HBASE-7423 and HBASE-7422
> since the fs.getConf() can be the unexpected one, we should avoid to use that 
> to get the root dir

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7439) HFileLink should not use the configuration from the Filesystem

2012-12-26 Thread Ted Yu (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13539733#comment-13539733
 ] 

Ted Yu commented on HBASE-7439:
---

Why is hdfs block distribution method removed ?

> HFileLink should not use the configuration from the Filesystem
> --
>
> Key: HBASE-7439
> URL: https://issues.apache.org/jira/browse/HBASE-7439
> Project: HBase
>  Issue Type: Sub-task
>  Components: Client, master, regionserver, snapshots, Zookeeper
>Affects Versions: hbase-6055
>Reporter: Matteo Bertozzi
>Assignee: Matteo Bertozzi
>Priority: Blocker
> Fix For: hbase-6055, 0.96.0
>
> Attachments: HBASE-7439-v0.patch
>
>
> This is related to HBASE-7423 and HBASE-7422
> since the fs.getConf() can be the unexpected one, we should avoid to use that 
> to get the root dir

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira