Skip to content

Commit 73d1ac1

Browse files
committed
ATLAS-4922: Addressed PR comments for Client-side Changes & Renamed Constants
1 parent 3e84ea7 commit 73d1ac1

File tree

5 files changed

+28
-9
lines changed

5 files changed

+28
-9
lines changed

atlas-examples/sample-app/src/main/java/org/apache/atlas/examples/sampleapp/AsyncImportApiExample.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,10 @@ public void testGetAsyncImportStatus() throws Exception {
7676
try {
7777
PList<AsyncImportStatus> statuses = client.getAsyncImportStatus();
7878

79-
System.out.println("All Async Import Statuses: " + statuses);
79+
System.out.println("All Async Import Statuses:");
80+
for (AsyncImportStatus status : statuses.getList()) {
81+
System.out.println(status);
82+
}
8083
} catch (AtlasServiceException e) {
8184
System.err.println("Failed to fetch all async import statuses: " + e.getMessage());
8285

client/client-v2/src/main/java/org/apache/atlas/AtlasClientV2.java

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -150,8 +150,6 @@ public class AtlasClientV2 extends AtlasBaseClient {
150150
// Async Import APIs
151151
private static final String ASYNC_IMPORT_URI = BASE_URI + "admin/async/import";
152152
private static final String ASYNC_IMPORT_STATUS_URI = BASE_URI + "admin/async/import/status";
153-
private static final String ASYNC_IMPORT_STATUS_BY_ID_URI = BASE_URI + "admin/async/import/status/";
154-
private static final String ASYNC_IMPORT_BY_ID_URI = BASE_URI + "admin/async/import/";
155153

156154
private static final String IMPORT_REQUEST_PARAMTER = "request";
157155
private static final String IMPORT_DATA_PARAMETER = "data";
@@ -1060,7 +1058,15 @@ public AtlasAsyncImportRequest importAsync(AtlasImportRequest request, InputStre
10601058
}
10611059

10621060
public PList<AsyncImportStatus> getAsyncImportStatus() throws AtlasServiceException {
1063-
return callAPI(API_V2.ASYNC_IMPORT_STATUS_ALL, new GenericType<PList<AsyncImportStatus>>() {}, null);
1061+
return getAsyncImportStatus(0, 50);
1062+
}
1063+
1064+
public PList<AsyncImportStatus> getAsyncImportStatus(Integer offset, Integer limit) throws AtlasServiceException {
1065+
MultivaluedMap<String, String> queryParams = new MultivaluedMapImpl();
1066+
queryParams.add("offset", String.valueOf(offset));
1067+
queryParams.add("limit", String.valueOf(limit));
1068+
1069+
return callAPI(API_V2.ASYNC_IMPORT_STATUS_ALL, new GenericType<PList<AsyncImportStatus>>() {}, queryParams);
10641070
}
10651071

10661072
public AtlasAsyncImportRequest getAsyncImportStatusById(String importId) throws AtlasServiceException {
@@ -1298,8 +1304,8 @@ public static class API_V2 extends API {
12981304
// Async Import APIs
12991305
public static final API_V2 ASYNC_IMPORT = new API_V2(ASYNC_IMPORT_URI, HttpMethod.POST, Response.Status.OK, MediaType.MULTIPART_FORM_DATA, MediaType.APPLICATION_JSON);
13001306
public static final API_V2 ASYNC_IMPORT_STATUS_ALL = new API_V2(ASYNC_IMPORT_STATUS_URI, HttpMethod.GET, Response.Status.OK);
1301-
public static final API_V2 ASYNC_IMPORT_STATUS_BY_ID = new API_V2(ASYNC_IMPORT_STATUS_BY_ID_URI + "%s", HttpMethod.GET, Response.Status.OK);
1302-
public static final API_V2 ABORT_ASYNC_IMPORT_BY_ID = new API_V2(ASYNC_IMPORT_BY_ID_URI + "%s", HttpMethod.DELETE, Response.Status.NO_CONTENT);
1307+
public static final API_V2 ASYNC_IMPORT_STATUS_BY_ID = new API_V2(ASYNC_IMPORT_STATUS_URI + "/%s", HttpMethod.GET, Response.Status.OK);
1308+
public static final API_V2 ABORT_ASYNC_IMPORT_BY_ID = new API_V2(ASYNC_IMPORT_URI + "/%s", HttpMethod.DELETE, Response.Status.NO_CONTENT);
13031309

13041310
// Glossary APIs
13051311
public static final API_V2 GET_ALL_GLOSSARIES = new API_V2(GLOSSARY_URI, HttpMethod.GET, Response.Status.OK);

common/src/main/java/org/apache/atlas/repository/Constants.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ public final class Constants {
233233
* Atlas Async Import vertex property keys.
234234
*/
235235
public static final String ATLAS_ASYNC_IMPORT_PREFIX = INTERNAL_PROPERTY_KEY_PREFIX + "AtlasAsyncImportRequest.";
236-
public static final String PROPERTY_KEY_RECEIVED_AT = encodePropertyKey(ATLAS_ASYNC_IMPORT_PREFIX + "receivedAt");
236+
public static final String PROPERTY_KEY_RECEIVED_TIME = encodePropertyKey(ATLAS_ASYNC_IMPORT_PREFIX + "receivedTime");
237237
public static final String PROPERTY_KEY_ASYNC_IMPORT_STATUS = encodePropertyKey(ATLAS_ASYNC_IMPORT_PREFIX + "status");
238238
public static final String PROPERTY_KEY_ASYNC_IMPORT_ID = encodePropertyKey(ATLAS_ASYNC_IMPORT_PREFIX + "importId");
239239

intg/src/main/java/org/apache/atlas/model/impexp/AsyncImportStatus.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,4 +63,14 @@ public String getImportRequestReceivedTime() {
6363
public String getImportRequestUser() {
6464
return importRequestUser;
6565
}
66+
67+
@Override
68+
public String toString() {
69+
return "AsyncImportStatus{" +
70+
"importId='" + importId + '\'' +
71+
", status='" + status + '\'' +
72+
", importRequestReceivedTime='" + importRequestReceivedTime + '\'' +
73+
", importRequestUser='" + importRequestUser + '\'' +
74+
'}';
75+
}
6676
}

repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasGraphUtilsV2.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@
6767
import static org.apache.atlas.repository.Constants.INDEX_SEARCH_VERTEX_PREFIX_DEFAULT;
6868
import static org.apache.atlas.repository.Constants.INDEX_SEARCH_VERTEX_PREFIX_PROPERTY;
6969
import static org.apache.atlas.repository.Constants.PROPAGATED_CLASSIFICATION_NAMES_KEY;
70-
import static org.apache.atlas.repository.Constants.PROPERTY_KEY_RECEIVED_AT;
70+
import static org.apache.atlas.repository.Constants.PROPERTY_KEY_RECEIVED_TIME;
7171
import static org.apache.atlas.repository.Constants.RELATIONSHIP_TYPE_PROPERTY_KEY;
7272
import static org.apache.atlas.repository.Constants.STATE_PROPERTY_KEY;
7373
import static org.apache.atlas.repository.Constants.SUPER_TYPES_PROPERTY_KEY;
@@ -565,7 +565,7 @@ public static List<String> findEntityPropertyValuesByTypeAndAttributes(AtlasGrap
565565
}
566566
}
567567

568-
query.orderBy(PROPERTY_KEY_RECEIVED_AT, ASC);
568+
query.orderBy(PROPERTY_KEY_RECEIVED_TIME, ASC);
569569

570570
List<String> propertyValues = new ArrayList<>();
571571

0 commit comments

Comments
 (0)