[
https://issues.apache.org/jira/browse/PIG-3623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13856907#comment-13856907
]
Rohini Palaniswamy commented on PIG-3623:
-----------------------------------------
We should fix the code and not the documentation as the expected behavior
should that be of the documentation. Since default is false, people don't
usually specify -loadKey false and if they specified they would immediately
realize as in this jira that it is not working correctly because the fields
returned are not expected, so backward compatibility is not a issue. Saw this
with -noWAL as well when fixing PIG-3632. Both options need to be fixed.
> Documentation for loadKey in HBaseStorage is incorrect
> ------------------------------------------------------
>
> Key: PIG-3623
> URL: https://issues.apache.org/jira/browse/PIG-3623
> Project: Pig
> Issue Type: Bug
> Reporter: Michael Stefaniak
>
> The documentation for HBaseStorage
> (http://pig.apache.org/docs/r0.12.0/func.html#HBaseStorage)
> says -loadKey=(true|false) Load the row key as the first value in every tuple
> returned from HBase (default=false)
> However, looking at the source
> (http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/hbase/HBaseStorage.java)
> it is just doing a check for the existence of this option
> loadRowKey_ = configuredOptions_.hasOption("loadKey");
> So setting -loadKey=false in the options string, still results in a true value
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
