[
https://issues.apache.org/jira/browse/HBASE-15447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15207138#comment-15207138
]
Misty Stanley-Jones commented on HBASE-15447:
---------------------------------------------
Thanks [~wchevreuil]. This looks pretty good. A little feedback for next time:
- Please create your patches using {{git format-patch}} rather than {{git
diff}}. This will preserve your commit message and make it much easier for us
to apply your patch. See http://hbase.apache.org/book.html#submitting.patches.
- Please be careful about trailing whitespace. Your text editor may have tools
to clean it up. I noticed a couple instances of this, but I'll clean it up upon
commit.
> Improve javadocs description for Delete methods
> -----------------------------------------------
>
> Key: HBASE-15447
> URL: https://issues.apache.org/jira/browse/HBASE-15447
> Project: HBase
> Issue Type: Improvement
> Components: documentation
> Reporter: Wellington Chevreuil
> Assignee: Wellington Chevreuil
> Priority: Minor
> Attachments: HBASE-15447.patch
>
>
> Current javadoc for Delete operation is a bit confusing. Even though initial
> section does describe the proper behaviour, where calling delete without
> specifying timestamp will only delete records from now to the past, this is
> not as clear in the method specific description
> {noformat}
> public Delete(byte[] row)
> Create a Delete operation for the specified row.
> If no further operations are done, this will delete everything associated
> with the specified row (all versions of all columns in all families).
> Parameters:
> row - row key
> {noformat}
> The description can lead to the conclusion that all versions will be deleted.
> But that's not true if a row has a future timestamp. Although this is a very
> unusual scenario (having a future timestamp), it should still be clarified.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
