You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Jeremy Karlson <je...@gmail.com> on 2013/11/28 08:32:40 UTC
Review Request 15906: Generic JDBC Sink

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

Review request for Flume.


Repository: flume-git


Description
-------

https://issues.apache.org/jira/browse/FLUME-2256


Diffs
-----

  flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkConfiguration.java 965d2c5 
  flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkType.java 8ce372d 
  flume-ng-dist/pom.xml 2d0ee47 
  flume-ng-sinks/flume-ng-jdbc-sink/pom.xml PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/AbstractJDBCSink.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/BodyParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/ByteArrayBodyParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/CustomParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/DateHeaderParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/DoubleHeaderParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/FloatHeaderParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/HeaderParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/IntHeaderParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/JDBCConnectionManager.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/LongHeaderParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/Parameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/PreparedStatementJDBCSink.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/PreparedStatementParser.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/StringBodyParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/main/java/org/apache/flume/sink/jdbc/StringHeaderParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/test/java/org/apache/flume/sink/jdbc/TestAbstractJDBCSink.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/test/java/org/apache/flume/sink/jdbc/TestJDBCConnectionManager.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/test/java/org/apache/flume/sink/jdbc/TestJDBCSinkCreation.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/test/java/org/apache/flume/sink/jdbc/TestParameter.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/test/java/org/apache/flume/sink/jdbc/TestPreparedStatementJDBCSink.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/test/java/org/apache/flume/sink/jdbc/TestPreparedStatementParser.java PRE-CREATION 
  flume-ng-sinks/flume-ng-jdbc-sink/src/test/resources/log4j.properties PRE-CREATION 
  flume-ng-sinks/pom.xml d03576b 
  pom.xml 267925f 

Diff: https://reviews.apache.org/r/15906/diff/


Testing
-------

Some end to end testing; different data types require further testing.


Thanks,

Jeremy Karlson