Re: Review Request: [rest] content transcoding

[Andrew Purtell <ap...@apache.org>]

> On 2010-10-30 22:24:29, stack wrote:
> > Looks good Andy.  Theres a bit of white space.  Try squash it on commit.  You don't have a test of the new facility.  Would that be hard to add?

Thanks for the review. I'll squash WS add a unit test before commit.


- Andrew


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/1136/#review1727
-----------------------------------------------------------


On 2010-10-30 01:29:22, Andrew Purtell wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://review.cloudera.org/r/1136/
> -----------------------------------------------------------
> 
> (Updated 2010-10-30 01:29:22)
> 
> 
> Review request for hbase.
> 
> 
> Summary
> -------
> 
> Introduce a table schema attribute for column families that instructs the REST gateway to perform input and/or output transcoding, with base64->binary (for GET) and vice versa (for PUT or POST) as the first supported option.
> 
> 
> This addresses bug HBASE-3120.
>     http://issues.apache.org/jira/browse/HBASE-3120
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/hadoop/hbase/rest/ExistsResource.java d83dc20 
>   src/main/java/org/apache/hadoop/hbase/rest/RegionsResource.java 54f2dc7 
>   src/main/java/org/apache/hadoop/hbase/rest/RowResource.java 1bf17b8 
>   src/main/java/org/apache/hadoop/hbase/rest/ScannerResource.java 5bb90ef 
>   src/main/java/org/apache/hadoop/hbase/rest/SchemaResource.java 7262229 
>   src/main/java/org/apache/hadoop/hbase/rest/TableResource.java 5f305db 
>   src/main/java/org/apache/hadoop/hbase/rest/transform/Base64.java PRE-CREATION 
>   src/main/java/org/apache/hadoop/hbase/rest/transform/NullTransform.java PRE-CREATION 
>   src/main/java/org/apache/hadoop/hbase/rest/transform/Transform.java PRE-CREATION 
> 
> Diff: http://review.cloudera.org/r/1136/diff
> 
> 
> Testing
> -------
> 
> Created a test table. Modified the HCD of a column family by adding a custom attribute of 'Transform$1' => '*:Base64'. Used curl to store the string "Testing 1, 2, 3" using content type application/octet-stream. Confirmed with shell the value was base64 encoded in table. Used curl to retrieve the value using accept header of application/octet-stream and confirmed the response body was the decoded string.
> 
> 
> Thanks,
> 
> Andrew
> 
>