----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71025/ -----------------------------------------------------------
(Updated July 24, 2019, 4:58 p.m.) Review request for atlas, Madhan Neethiraj, Nikhil Bonte, Nixon Rodrigues, and Sarath Subramanian. Changes ------- Updates include: - Added pre-commit build details. Bugs: ATLAS-3320 https://issues.apache.org/jira/browse/ATLAS-3320 Repository: atlas Description ------- **Approach** - Use existing producer-consumer (PC) framework. - Modify _BulkImporterImpl_ to implement _WorkItemConsumer_. - Add support for configuring number of workers and batch size within _AtlasImportRequest_. _AtlasImportRequest_ ``` { "options": { "numWorkers": 8, "batchSize": 25 } } ``` **CURL** ``` curl -v -X POST -u admin:admin -H "Content-Type: multipart/form-data" -H "Cache-Control: no-cache" -F request=@./import-options.json -F data=@./Default-3-pre.zip http://localhost:21000/api/atlas/admin/import ``` Diffs ----- graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraph.java 613a714ff intg/src/main/java/org/apache/atlas/model/impexp/AtlasImportRequest.java 0b3ede93f repository/src/main/java/org/apache/atlas/repository/impexp/ImportService.java 3ded79842 repository/src/main/java/org/apache/atlas/repository/store/graph/v2/BulkImporterImpl.java 2f330c093 Diff: https://reviews.apache.org/r/71025/diff/2/ Testing (updated) ------- **Unit tests** Existing tests. **Functional tests** - Verified import for pre-1.0 and post-1.0 exported ZIP files. **Pre-commit** https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/1292 **Volume tests** - Measure performance with large data. +----------+----------+----------+------------------------+ | File | Before | After | Configuration | +----------+----------+----------+------------------------+ | smalldb | 6 min | 3 min | Shards: 4, Threads: 8 | | (2.2 MB) | | | | +----------+----------+----------+------------------------+ | largedb | 3 hrs | 20 mins | Shards: 4, Threads: 16 | | (40 MB) | | | | +----------+----------+----------+------------------------+ Thanks, Ashutosh Mestry