Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Flume, mail # dev - Review Request: BasicTransactionSemantics should avoid throw-ing from close()


Copy link to this message
-
Review Request: BasicTransactionSemantics should avoid throw-ing from close()
Roshan Naik 2013-04-09, 03:21

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

Review request for Flume.
Description
-------

Loggin error instead of throwing exception in cases where close() is called w/o rollback or abort being called.
This addresses bug FLUME-1321.
    https://issues.apache.org/jira/browse/FLUME-1321
Diffs
-----

  flume-ng-core/src/main/java/org/apache/flume/channel/BasicTransactionSemantics.java 403cbca

Diff: https://reviews.apache.org/r/10362/diff/
Testing
-------
Thanks,

Roshan Naik

+
Mike Percy 2013-04-16, 22:51
+
Mike Percy 2013-04-16, 22:52
+
Roshan Naik 2013-04-17, 00:44
+
Mike Percy 2013-04-17, 20:39
+
Brock Noland 2013-04-09, 20:31
+
Hari Shreedharan 2013-04-09, 20:46
+
Brock Noland 2013-04-09, 20:54
+
Roshan Naik 2013-04-09, 23:52
+
Brock Noland 2013-04-10, 18:19
+
Roshan Naik 2013-04-10, 18:28
+
Brock Noland 2013-04-10, 18:53
+
Brock Noland 2013-04-10, 19:09