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

Rohini Palaniswamy commented on PIG-2997:
-----------------------------------------

Prashant,
   I did not mean that you need to write a unit test for this. It would be 
overkill :). Sorry for not being clear. There are already unit tests which use 
ConfigurationUtil.toProperties() - TestHBaseStorage.beforeTest() and 
TestFRJoin.FRJoin.setUpHashTable(). It would be good if you can change those 
existing tests to use your new API instead of a new test.
                
> Provide a convenience constructor on PigServer that accepts Configuration
> -------------------------------------------------------------------------
>
>                 Key: PIG-2997
>                 URL: https://issues.apache.org/jira/browse/PIG-2997
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Prashant Kommireddi
>            Assignee: Prashant Kommireddi
>             Fix For: 0.11
>
>         Attachments: PIG-2997_1.patch, PIG-2997.patch
>
>
> PigServer currently has Properties based constructor. Hadoop in general deals 
> with Configuration and it would be good to have a PigServer constructor that 
> accepts the same. With this, user does not have to worry about creating 
> Properties object out of conf and can simply invoke this new constructor.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to