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

[email protected] commented on FLUME-1183:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5073/#review7759
-----------------------------------------------------------


I've taken a look over this too and don't really have anything to add to brocks 
comments.
I've also compiled it and run the tests and everything seems good though needed 
to run mvn install first, and set my umask to 022 (otherwise the minicluster 
datanode would complain and abort everything).

I don't see anything wrong with it, and once you've added the changes suggested 
by brock I'll put it through a quick spin on a test machine and it should 
probably be good to go.

- Juhani


On 2012-05-09 03:04:07, Hari Shreedharan wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/5073/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-05-09 03:04:07)
bq.  
bq.  
bq.  Review request for Flume.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Hbase sink.
bq.  
bq.  
bq.  This addresses bug FLUME-1183.
bq.      https://issues.apache.org/jira/browse/FLUME-1183
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    
flume-ng-sinks/flume-ng-hbase-sink/src/test/java/org/apache/flume/sink/hbase/TestHBaseSink.java
 PRE-CREATION 
bq.    flume-ng-sinks/pom.xml acb3087 
bq.    pom.xml 8c11a2d 
bq.    flume-ng-dist/pom.xml 5bdcfe7 
bq.    flume-ng-sinks/flume-ng-hbase-sink/pom.xml PRE-CREATION 
bq.    
flume-ng-sinks/flume-ng-hbase-sink/src/main/java/org/apache/flume/sink/hbase/HBaseSink.java
 PRE-CREATION 
bq.    bin/flume-ng 0108997 
bq.  
bq.  Diff: https://reviews.apache.org/r/5073/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Unit tests added
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Hari
bq.  
bq.


                
> Implement an HBase Sink which supports table level access
> ---------------------------------------------------------
>
>                 Key: FLUME-1183
>                 URL: https://issues.apache.org/jira/browse/FLUME-1183
>             Project: Flume
>          Issue Type: New Feature
>          Components: Sinks+Sources
>    Affects Versions: v1.2.0
>            Reporter: Hari Shreedharan
>            Assignee: Hari Shreedharan
>             Fix For: v1.2.0
>
>
> This is what I intend to do:
> * Insert the row key from event headers. Pick the column family and column 
> from configuration, not from headers. 
> * Allow configuration to specify default for row key. If no row key exists in 
> header, then take the default value from the configuration. I don't want to 
> dump everything into the table with the same row key. 
> * I don't intend to support multiple tables, column families or columns in 
> the same sink. We can use multiplexing channel selector to use different 
> sinks for different tables/columns.
> I know the existence of another jira for porting the HBase sink from OG, but 
> didn't see any activity for a while on that. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to