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

ASF GitHub Bot commented on FLINK-2195:
---------------------------------------

Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/828#discussion_r32303124
  
    --- Diff: 
flink-java/src/main/java/org/apache/flink/api/java/hadoop/mapreduce/HadoopInputFormatBase.java
 ---
    @@ -131,6 +136,11 @@ public BaseStatistics getStatistics(BaseStatistics 
cachedStats) throws IOExcepti
                        throw new RuntimeException(e);
                }
     
    +           // Configure input format for splits
    +           if (mapreduceInputFormat instanceof Configurable) {
    --- End diff --
    
    I think this is not necessary, because the JM calls `configure()` before 
requesting InputSplits.


> Set Configuration for Configurable Hadoop InputFormats
> ------------------------------------------------------
>
>                 Key: FLINK-2195
>                 URL: https://issues.apache.org/jira/browse/FLINK-2195
>             Project: Flink
>          Issue Type: New Feature
>          Components: Core
>    Affects Versions: master
>            Reporter: Ufuk Celebi
>            Assignee: Ufuk Celebi
>
> Configurable Hadoop InputFormats like HBase's mapreduce.TableInputFormat can 
> require the Configuration to be set in order to work correctly.
> The mapreduce.TableInputFormat (HBase 1.0.1.1) for example fails with a NPE 
> because of this.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to