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

(Updated Dec. 4, 2017, 12:09 p.m.)


Review request for Aurora, David McLaughlin, Jordan Ly, and Stephan Erb.


Repository: aurora


Description
-------

Introduces a `Persistence` implementation that uses a SQL database via JDBC.  
I've opted to lean towards MySQL SQL dialect, as unfortunately there are vendor 
differences for even the very simple SQL used here.

I chose [HikariCP](https://github.com/brettwooldridge/HikariCP) to serve as a 
`DataSource` (connection pool) implementation.  We don't really need much out 
of a connection pool aside from general connection lifecycle management (i.e. 
not for concurrency).  I chose this library based on recent development 
activity and several positive comparisons to other pools.

Note that the implementation is not yet wired into the scheduler application.  
That will come in a follow-up.


Diffs (updated)
-----

  build.gradle af119910e84c48f75f2573ababcfa287c3b986fc 
  config/spotbugs/excludeFilter.xml 51790cce8d9047e40741f05ee55af15dbdc3065e 
  
src/main/java/org/apache/aurora/scheduler/storage/durability/DurableStorage.java
 85b2113631586f43d854c4d2812f43b7b864d452 
  src/main/java/org/apache/aurora/scheduler/storage/durability/Persistence.java 
9eb862c01bf451252bfbcc7a2eac60d2c965c9f0 
  src/main/java/org/apache/aurora/scheduler/storage/log/LogPersistence.java 
e70e6051582ca90ae72014626b983bbf4b8d5b48 
  src/main/java/org/apache/aurora/scheduler/storage/sql/SqlPersistence.java 
PRE-CREATION 
  
src/main/java/org/apache/aurora/scheduler/storage/sql/SqlPersistenceModule.java 
PRE-CREATION 
  src/main/resources/org/apache/aurora/scheduler/storage/sql/schema.sql 
PRE-CREATION 
  src/test/java/org/apache/aurora/scheduler/storage/sql/SqlPersistenceTest.java 
PRE-CREATION 


Diff: https://reviews.apache.org/r/64288/diff/2/

Changes: https://reviews.apache.org/r/64288/diff/1-2/


Testing
-------


Thanks,

Bill Farner

Reply via email to