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

(Updated Jan. 30, 2018, 3:29 p.m.)


Review request for atlas and Mandy Chessell.


Changes
-------

update after 30-01 review


Repository: atlas


Description
-------

The OCF Database Connector is the subclass of OCF Connector and it is designed 
especially for connection to database to retrieve data. 
Here we implement a connector for Gaian (GaianOCFConnector) as an example for 
using OCF Database Connector. It is related to Open Connector Framework. The 
JIRA can be found https://issues.apache.org/jira/browse/ATLAS-2298


Diffs
-----

  ocfdb/README.md  
  ocfdb/pom.xml  
  ocfdb/src/main/java/org/apache/atlas/ocfdb/connectors/OCFDataProvider.java  
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/connectors/OCFDatabaseConnection.java
  
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/connectors/OCFDatabaseConnector.java 
 
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/connectors/OCFDatabaseConnectorProviderBase.java
  
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/connectors/gaian/GaianOCFConnector.java
  
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/connectors/gaian/GaianOCFConnectorProvider.java
  
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/ffdc/DatabaseAccessCheckedException.java
  
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/ffdc/OCFDatabaseCheckedExceptionBase.java
  
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/ffdc/OCFDatabaseConnectorErrorCode.java
  
  
ocfdb/src/main/java/org/apache/atlas/ocfdb/ffdc/OCFDatabaseConnectorRuntimeException.java
  


Diff: https://reviews.apache.org/r/65123/diff/5/


Testing
-------

create an instance of the Connector and use getData() function. Gaian has to be 
set up in advance.


File Attachments (updated)
----------------

ATLAS-2298-update-patch-based-on-reviews.patch
  
https://reviews.apache.org/media/uploaded/files/2018/01/29/e228728b-3ac4-49a9-8b44-bb3ff2a04052__ATLAS-2298-update-patch-based-on-reviews.patch
ATLAS-2298-update-patch-based-on-reviews.patch
  
https://reviews.apache.org/media/uploaded/files/2018/01/29/425dded1-e882-4f5b-902b-acea0acec74d__ATLAS-2298-update-patch-based-on-reviews.patch
ATLAS-2298-update-the-patch-29-1.patch
  
https://reviews.apache.org/media/uploaded/files/2018/01/29/4c6d596c-dd34-4cf6-8fe0-33258e9965a6__ATLAS-2298-update-the-patch-29-1.patch
ATLAS-2298-update-the-patch-29-1.patch
  
https://reviews.apache.org/media/uploaded/files/2018/01/29/db1c4085-05c8-41d7-9367-274e5dadfa1c__ATLAS-2298-update-the-patch-29-1.patch
ATLAS-2298-30-Jan-review-verision.patch
  
https://reviews.apache.org/media/uploaded/files/2018/01/30/b4282020-0ef5-41bc-82d2-a0e0d64d592b__ATLAS-2298-30-Jan-review-verision.patch


Thanks,

Yao Li

Reply via email to