-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53423/#review156499
-----------------------------------------------------------
Ship it!
It seems good. A couple of high level comments:
- There is some mix of naming with regards to transaction isolation level. It seems "transaction level" and "isolation level" are used interchangeably in the patch. I'd choose on and go with it for consistency.
- Transaction isolation level is only an issue with certain databases (PDW and SQL Server for example). I haven't looked recently, but is this a problem with other databases other than SQL Server esque DBs? You could probably move forward with this patch and think about this later.

- Abraham Elmahrek
On Nov. 10, 2016, 2:13 p.m., Attila Szabo wrote:

NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB