-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2937/
-----------------------------------------------------------
Review request for Flume and Prasad Mujumdar.
Summary
-------
Added support for specifying the capacity of the JDBC channel. If a non-zero
positive value is specified, the channel enforces it as the limit on total
number of events persisted at any time. If a zero/negative/unspecified value is
provided, the channel operates without any limit. Added a test case as well to
exercise this functionality.
Another change that I am sliding in is reducing the overall number of events
that tests exercise - from about 2000 to close to 200. This will greatly reduce
the amount of time the tests take when running on the JDBC channel module. The
test functionality remains the same.
This addresses bug FLUME-820.
https://issues.apache.org/jira/browse/FLUME-820
Diffs
-----
flume-ng-channels/flume-jdbc-channel/src/main/java/org/apache/flume/channel/jdbc/ConfigurationConstants.java
7e0b3ac
flume-ng-channels/flume-jdbc-channel/src/main/java/org/apache/flume/channel/jdbc/impl/DerbySchemaHandler.java
7b29c7d
flume-ng-channels/flume-jdbc-channel/src/main/java/org/apache/flume/channel/jdbc/impl/JdbcChannelProviderImpl.java
c1bc400
flume-ng-channels/flume-jdbc-channel/src/main/java/org/apache/flume/channel/jdbc/impl/MySQLSchemaHandler.java
1065ef9
flume-ng-channels/flume-jdbc-channel/src/main/java/org/apache/flume/channel/jdbc/impl/SchemaHandler.java
68946c9
flume-ng-channels/flume-jdbc-channel/src/test/java/org/apache/flume/channel/jdbc/MockEventUtils.java
78eda51
flume-ng-channels/flume-jdbc-channel/src/test/java/org/apache/flume/channel/jdbc/TestDerbySchemaHandlerQueries.java
9ce6d16
flume-ng-channels/flume-jdbc-channel/src/test/java/org/apache/flume/channel/jdbc/TestJdbcChannelProvider.java
4c3012d
Diff: https://reviews.apache.org/r/2937/diff
Testing
-------
Ran all JDBC channel tests.
Thanks,
Arvind
