apex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (APEXMALHAR-2008) Create hdfs file input module
Date Wed, 09 Mar 2016 12:33:40 GMT

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

ASF GitHub Bot commented on APEXMALHAR-2008:
--------------------------------------------

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

    https://github.com/apache/incubator-apex-malhar/pull/207#discussion_r55511509
  
    --- Diff: library/src/main/java/com/datatorrent/lib/io/block/HDFSBlockReader.java ---
    @@ -0,0 +1,78 @@
    +/**
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + *
    + *   http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing,
    + * software distributed under the License is distributed on an
    + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    + * KIND, either express or implied.  See the License for the
    + * specific language governing permissions and limitations
    + * under the License.
    + */
    +package com.datatorrent.lib.io.block;
    +
    +import java.io.IOException;
    +import java.net.URI;
    +
    +import org.apache.hadoop.fs.FileSystem;
    +
    +import com.google.common.base.Splitter;
    +
    +public class HDFSBlockReader extends FSSliceReader
    +{
    +  protected String uri;
    +
    +  @Override
    +  protected FileSystem getFSInstance() throws IOException
    +  {
    +    return FileSystem.newInstance(URI.create(uri), configuration);
    +  }
    +
    +  /**
    +   * Sets the uri
    +   *
    +   * @param uri
    +   */
    +  public void setUri(String uri)
    +  {
    +    this.uri = convertSchemeToLowerCase(uri);
    +  }
    +
    +  public String getUri()
    +  {
    +    return uri;
    +  }
    +
    +  /**
    +   * Converts Scheme part of the URI to lower case. Multiple URI can be comma separated.
If no scheme is there, no
    +   * change is made.
    +   * 
    +   * @param
    +   * @return String with scheme part as lower case
    +   */
    +  private static String convertSchemeToLowerCase(String uri)
    --- End diff --
    
    Is there a test case covering  this?


> Create hdfs file input module 
> ------------------------------
>
>                 Key: APEXMALHAR-2008
>                 URL: https://issues.apache.org/jira/browse/APEXMALHAR-2008
>             Project: Apache Apex Malhar
>          Issue Type: Task
>            Reporter: Priyanka Gugale
>            Assignee: Priyanka Gugale
>            Priority: Minor
>   Original Estimate: 72h
>  Remaining Estimate: 72h
>
> To read HDFS files in parallel using Apex we normally use FileSplitter and FileReader
module. It would be a good idea to combine those operators as a unit in module. Having a module
will give us readily usable set of operators to read HDFS files. 



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

Mime
View raw message