[jira] [Commented] (FLUME-3128) Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible changes in hadoop-hdfs lib.

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLUME-3128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16094259#comment-16094259
 ] 

ASF GitHub Bot commented on FLUME-3128:
---

Github user mcsanady closed the pull request at:

https://github.com/apache/flume/pull/146


> Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible changes in 
> hadoop-hdfs lib.
> ---
>
> Key: FLUME-3128
> URL: https://issues.apache.org/jira/browse/FLUME-3128
> Project: Flume
>  Issue Type: Bug
>Reporter: Miklos Csanady
>Assignee: Miklos Csanady
>
> Our test in 
> flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java
> Uses the getLeaseByPath(String src) method of Lease class which is to be 
> replaced by getLeaseForPath(Namenode n, String src) method.
> This change is to make Flume compatible after the hadoop change is made 
> effective.
> See https://issues.apache.org/jira/browse/HDFS-6757



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (FLUME-3128) Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible changes in hadoop-hdfs lib.

[Miklos Csanady (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLUME-3128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16093232#comment-16093232
 ] 

Miklos Csanady commented on FLUME-3128:
---

I won't work. NameNodeAdapter class is not present in the hadoop used by flume 
in trunk.

> Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible changes in 
> hadoop-hdfs lib.
> ---
>
> Key: FLUME-3128
> URL: https://issues.apache.org/jira/browse/FLUME-3128
> Project: Flume
>  Issue Type: Bug
>Reporter: Miklos Csanady
>Assignee: Miklos Csanady
>
> Our test in 
> flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java
> Uses the getLeaseByPath(String src) method of Lease class which is to be 
> replaced by getLeaseForPath(Namenode n, String src) method.
> This change is to make Flume compatible after the hadoop change is made 
> effective.
> See https://issues.apache.org/jira/browse/HDFS-6757



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (FLUME-3128) Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible changes in hadoop-hdfs lib.

[Miklos Csanady (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLUME-3128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16093207#comment-16093207
 ] 

Miklos Csanady commented on FLUME-3128:
---

[~denes] can you take a look at my pull request: 
https://github.com/apache/flume/pull/146 ?

> Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible changes in 
> hadoop-hdfs lib.
> ---
>
> Key: FLUME-3128
> URL: https://issues.apache.org/jira/browse/FLUME-3128
> Project: Flume
>  Issue Type: Bug
>Reporter: Miklos Csanady
>Assignee: Miklos Csanady
>
> Our test in 
> flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java
> Uses the getLeaseByPath(String src) method of Lease class which is to be 
> replaced by getLeaseForPath(Namenode n, String src) method.
> This change is to make Flume compatible after the hadoop change is made 
> effective.
> See https://issues.apache.org/jira/browse/HDFS-6757



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (FLUME-3128) Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible changes in hadoop-hdfs lib.

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/FLUME-3128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16093201#comment-16093201
 ] 

ASF GitHub Bot commented on FLUME-3128:
---

GitHub user mcsanady opened a pull request:

https://github.com/apache/flume/pull/146

FLUME-3128: Fix TestHDFSEventSinkOnMiniCluster.java due to incompatib…

…le changes in hadoop-hdfs lib.

Our test in 
flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java
Uses the getLeaseByPath(String src) method of Lease class which is to be 
replaced by getLeaseForPath(Namenode n, String src) method.

This change is to make Flume compatible after the hadoop change is made 
effective.

See https://issues.apache.org/jira/browse/HDFS-6757

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mcsanady/flume FLUME-3128

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/flume/pull/146.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #146


commit 20ec70a896f2e869a78ade8db2023c4c3bb96c32
Author: Denes Arvay 
Date:   2017-05-02T13:05:23Z

FLUME-3128: Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible 
changes in hadoop-hdfs lib.

Our test in 
flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java
Uses the getLeaseByPath(String src) method of Lease class which is to be 
replaced by getLeaseForPath(Namenode n, String src) method.

This change is to make Flume compatible after the hadoop change is made 
effective.

See https://issues.apache.org/jira/browse/HDFS-6757




> Fix TestHDFSEventSinkOnMiniCluster.java due to incompatible changes in 
> hadoop-hdfs lib.
> ---
>
> Key: FLUME-3128
> URL: https://issues.apache.org/jira/browse/FLUME-3128
> Project: Flume
>  Issue Type: Bug
>Reporter: Miklos Csanady
>Assignee: Miklos Csanady
>
> Our test in 
> flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java
> Uses the getLeaseByPath(String src) method of Lease class which is to be 
> replaced by getLeaseForPath(Namenode n, String src) method.
> This change is to make Flume compatible after the hadoop change is made 
> effective.
> See https://issues.apache.org/jira/browse/HDFS-6757



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)