[ 
https://issues.apache.org/jira/browse/HADOOP-16294?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Olson updated HADOOP-16294:
----------------------------------
    Description: 
In the DistCp class, the context is private with no getter method allowing 
retrieval by subclasses. So a subclass would need to save its own copy of the 
inputOptions supplied to its constructor and reconstruct the context if it 
wishes to override the createInputFileListing method with logic similar to the 
original implementation, i.e. calling CopyListing#buildListing with a path and 
context.

I propose adding to DistCp this method,

{noformat}
  protected DistCpContext getContext() {
    return context;
  }
{noformat}

  was:
In the DistCp class, the DistCpOptions are private with no getter method 
allowing retrieval by subclasses. So a subclass would need to save its own copy 
of the inputOptions supplied to its constructor, if it wishes to override the 
createInputFileListing method with logic similar to the original 
implementation, i.e. calling CopyListing#buildListing with a path and input 
options.

I propose adding to DistCp this method,

{noformat}
  protected DistCpOptions getInputOptions() {
    return inputOptions;
  }
{noformat}


> Enable access to input options by DistCp subclasses
> ---------------------------------------------------
>
>                 Key: HADOOP-16294
>                 URL: https://issues.apache.org/jira/browse/HADOOP-16294
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: tools/distcp
>            Reporter: Andrew Olson
>            Assignee: Andrew Olson
>            Priority: Trivial
>
> In the DistCp class, the context is private with no getter method allowing 
> retrieval by subclasses. So a subclass would need to save its own copy of the 
> inputOptions supplied to its constructor and reconstruct the context if it 
> wishes to override the createInputFileListing method with logic similar to 
> the original implementation, i.e. calling CopyListing#buildListing with a 
> path and context.
> I propose adding to DistCp this method,
> {noformat}
>   protected DistCpContext getContext() {
>     return context;
>   }
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to