jackhalfalltrades commented on code in PR #307:
URL: https://github.com/apache/atlas/pull/307#discussion_r1995972069


##########
webapp/src/main/java/org/apache/atlas/web/resources/AdminResource.java:
##########
@@ -679,6 +681,105 @@ public AtlasImportResult importData(@DefaultValue("{}") 
@FormDataParam("request"
         return result;
     }
 
+    @POST
+    @Path("/asyncImport")
+    @Produces(Servlets.JSON_MEDIA_TYPE)
+    @Consumes(MediaType.MULTIPART_FORM_DATA)
+    public AtlasAsyncImportRequest importAsync(@DefaultValue("{}") 
@FormDataParam("request") String jsonData,
+                                               @FormDataParam("data") 
InputStream inputStream) throws AtlasBaseException {
+        if (LOG.isDebugEnabled()) {
+            LOG.debug("==> AdminResource.importAsync(jsonData={}, 
inputStream={})", jsonData, (inputStream != null));
+        }
+        AtlasAuthorizationUtils.verifyAccess(new 
AtlasAdminAccessRequest(AtlasPrivilege.ADMIN_IMPORT), "asyncImportData");
+        AtlasAsyncImportRequest asyncImportRequest = null;
+        boolean preventMultipleRequests = true;

Review Comment:
   @mneethiraj This was copied from existing import - line 645. Do we need to 
update on both places ? Also will we not preventMultipleReuests, in the the 
request / request options is null but we still have valid import file 
inputstream to work with ?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@atlas.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to