[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200446#comment-13200446
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Mapreduce-trunk #979 (See
[https://builds.apache.org/job/Hadoop-Mapreduce-trunk/979/])
HDFS-2889. getNumCurrentReplicas is package private but should be public
on0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240412
Files :
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200425#comment-13200425
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Mapreduce-0.23-Build #181 (See
[https://builds.apache.org/job/Hadoop-Mapreduce-0.23-Build/181/])
HDFS-2889. getNumCurrentReplicas is package private but should be public on
0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240415
Files :
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200405#comment-13200405
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Hdfs-0.23-Build #159 (See
[https://builds.apache.org/job/Hadoop-Hdfs-0.23-Build/159/])
HDFS-2889. getNumCurrentReplicas is package private but should be public on
0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240415
Files :
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200392#comment-13200392
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Hdfs-trunk #946 (See
[https://builds.apache.org/job/Hadoop-Hdfs-trunk/946/])
HDFS-2889. getNumCurrentReplicas is package private but should be public
on0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240412
Files :
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200223#comment-13200223
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Mapreduce-trunk-Commit #1670 (See
[https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/1670/])
HDFS-2889. getNumCurrentReplicas is package private but should be public
on0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240412
Files :
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200202#comment-13200202
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Mapreduce-0.23-Commit #495 (See
[https://builds.apache.org/job/Hadoop-Mapreduce-0.23-Commit/495/])
HDFS-2889. getNumCurrentReplicas is package private but should be public on
0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240415
Files :
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200197#comment-13200197
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Hdfs-0.23-Commit #472 (See
[https://builds.apache.org/job/Hadoop-Hdfs-0.23-Commit/472/])
HDFS-2889. getNumCurrentReplicas is package private but should be public on
0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240415
Files :
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200193#comment-13200193
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Hdfs-trunk-Commit #1726 (See
[https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/1726/])
HDFS-2889. getNumCurrentReplicas is package private but should be public
on0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240412
Files :
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200187#comment-13200187
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Common-0.23-Commit #482 (See
[https://builds.apache.org/job/Hadoop-Common-0.23-Commit/482/])
HDFS-2889. getNumCurrentReplicas is package private but should be public on
0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240415
Files :
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/branches/branch-0.23/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13200190#comment-13200190
]
Hudson commented on HDFS-2408:
--
Integrated in Hadoop-Common-trunk-Commit #1655 (See
[https://builds.apache.org/job/Hadoop-Common-trunk-Commit/1655/])
HDFS-2889. getNumCurrentReplicas is package private but should be public
on0.23 (see HDFS-2408). Contributed by Gregory Chanan.
atm : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1240412
Files :
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
*
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DFSOutputStream.java
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13199292#comment-13199292
]
Aaron T. Myers commented on HDFS-2408:
--
bq. Can we do the same thing for 0.23 and trunk? I'm happy to open a new JIRA
and submit a patch.
I see no reason we can't do the same for 0.23 and trunk. Please file a new
JIRA/post a patch and I'll be happy to take a look at it ASAP.
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13199103#comment-13199103
]
Gregory Chanan commented on HDFS-2408:
--
Can we do the same thing for 0.23 and trunk? I'm happy to open a new JIRA and
submit a patch.
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 1.0.0
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Fix For: 0.20.205.0
>
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13121764#comment-13121764
]
Hadoop QA commented on HDFS-2408:
-
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12497951/hdfs-2408.txt
against trunk revision .
+1 @author. The patch does not contain any @author tags.
-1 tests included. The patch doesn't appear to include any new or modified
tests.
Please justify why no new tests are needed for this
patch.
Also please list what manual steps were performed to
verify this patch.
-1 patch. The patch command could not apply the patch.
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/1343//console
This message is automatically generated.
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 0.20.205.1
>Reporter: stack
>Assignee: stack
>Priority: Blocker
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13121761#comment-13121761
]
Suresh Srinivas commented on HDFS-2408:
---
Can you please commit this at the earlier. I can do that if you are busy. This
will help get rc2 out sooner.
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 0.20.205.1
>Reporter: stack
>Assignee: stack
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13121760#comment-13121760
]
Suresh Srinivas commented on HDFS-2408:
---
Aaron, this is 205 patch. Are you starting Jenkins pre-commit build?
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 0.20.205.1
>Reporter: stack
>Assignee: stack
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-2408) DFSClient#getNumCurrentReplicas is package private in 205 but public in branch-0.20-append
[
https://issues.apache.org/jira/browse/HDFS-2408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13121759#comment-13121759
]
Aaron T. Myers commented on HDFS-2408:
--
D'oh! I forgot that Jenkins isn't going to be able to apply the patch since the
patch is for 0.20-security. :)
Regardless, the patch is trivial. I'll commit it tomorrow unless there are any
objections in the mean time.
> DFSClient#getNumCurrentReplicas is package private in 205 but public in
> branch-0.20-append
> --
>
> Key: HDFS-2408
> URL: https://issues.apache.org/jira/browse/HDFS-2408
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs client
>Affects Versions: 0.20.205.1
>Reporter: stack
>Assignee: stack
> Attachments: hdfs-2408.txt
>
>
> The below commit broke hdfs-826 for hbase in 205 rc1. It changes the
> accessiblity from public to package private on getNumCurrentReplicas and now
> current shipping hbase's at least cannot get at this method.
> {code}
> Revision 1174483 - (view) (download) (annotate) - [select for diffs]
> Modified Fri Sep 23 01:30:18 2011 UTC (13 days, 4 hours ago) by szetszwo
> File length: 136876 byte(s)
> Diff to previous 1174479 (colored)
> svn merge -c 1171137 from branch-0.20-security for HDFS-2333.
> {code}
> Here is diff between above change and one just previous:
> http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-205/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?view=diff&r1=1174479&r2=1174483&diff_format=u
> This is a critical facility for us.
> It seems like making this one method public again is all thats needed. I can
> make a patch like the below:
> diff --git a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> index b9cb053..39955c9 100644
> --- a/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> +++ b/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
> @@ -3569,7 +3569,7 @@ public class DFSClient implements FSConstants,
> java.io.Closeable {
> * block is not yet allocated, then this API will return 0 because there
> are
> * no replicas in the pipeline.
> */
> -int getNumCurrentReplicas() throws IOException {
> +public int getNumCurrentReplicas() throws IOException {
>synchronized(dataQueue) {
> if (nodes == null) {
>return blockReplication;
> Can we get this into RC2?
> Thanks,
> St.Ack
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
