[
https://issues.apache.org/jira/browse/HBASE-762?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jim Kellerman resolved HBASE-762.
---------------------------------
Resolution: Fixed
Committed to branch and trunk. Thanks for the patch JD!
> deleteFamily takes timestamp, should only take row and family. Javadoc
> describes both cases but only implements the timestamp case.
> ------------------------------------------------------------------------------------------------------------------------------------
>
> Key: HBASE-762
> URL: https://issues.apache.org/jira/browse/HBASE-762
> Project: Hadoop HBase
> Issue Type: Bug
> Components: client, ipc, regionserver
> Affects Versions: 0.2.0, 0.2.1
> Reporter: Jonathan Gray
> Assignee: Izaak Rubin
> Priority: Minor
> Fix For: 0.2.1, 0.18.0
>
> Attachments: hbase-762.patch
>
>
> The three version of deleteFamily in client.HTable (Text, String, byte[])
> have varying descriptions about whether they take timestamps or not.
> public void deleteFamily(org.apache.hadoop.io.Text row,
> org.apache.hadoop.io.Text family, long timestamp) throws IOException
> Delete all cells for a row with matching column family at all timestamps.
> public void deleteFamily(String row, String family, long timestamp) throws
> IOException
> Delete all cells for a row with matching column family at all timestamps.
> public void deleteFamily(byte[] row, byte[] family, long timestamp) throws
> IOException
> Delete all cells for a row with matching column family with timestamps
> less than or equal to timestamp.
> These will become:
> public void deleteFamily(org.apache.hadoop.io.Text row,
> org.apache.hadoop.io.Text family) throws IOException
> Delete all cells for a row with matching column family at all timestamps.
> public void deleteFamily(String row, String family) throws IOException
> Delete all cells for a row with matching column family at all timestamps.
> public void deleteFamily(byte[] row, byte[] family) throws IOException
> Delete all cells for a row with matching column family at all timestamps.
> Per Jean-Daniel's comment, deleteAll should then not permit families. I'm
> unsure whether this is currently allowed or not, but the documentation must
> be updated either way.
> Will post patch after more thorough testing.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
