Skip to content

Commit c3f911e

Browse files
Merge pull request #245 from skyflowapi/SK-2302-multi-table-insert-support-in-v3-sdk
SK-2302 multi table insert support in v3 sdk
2 parents 7b83641 + 339d8f7 commit c3f911e

File tree

8 files changed

+23
-25
lines changed

8 files changed

+23
-25
lines changed

v3/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
</parent>
1212

1313
<artifactId>skyflow-java</artifactId>
14-
<version>2.0.0-dev.a742b35</version>
14+
<version>2.0.0-dev.3a80c07</version>
1515
<packaging>jar</packaging>
1616
<name>${project.groupId}:${project.artifactId}</name>
1717
<description>Skyflow V3 SDK for the Java programming language</description>

v3/src/main/java/com/skyflow/VaultClient.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
import com.skyflow.config.Credentials;
77
import com.skyflow.config.VaultConfig;
8-
import com.skyflow.enums.UpdateType;
8+
import com.skyflow.enums.UpsertType;
99
import com.skyflow.errors.ErrorCode;
1010
import com.skyflow.errors.ErrorMessage;
1111
import com.skyflow.errors.SkyflowException;
@@ -145,9 +145,9 @@ protected InsertRequest getBulkInsertRequestBody(com.skyflow.vault.data.InsertRe
145145
if (record.getUpsert() != null && !record.getUpsert().isEmpty()){
146146
if (record.getUpsertType() != null) {
147147
EnumUpdateType updateType = null;
148-
if (record.getUpsertType() == UpdateType.REPLACE) {
148+
if (record.getUpsertType() == UpsertType.REPLACE) {
149149
updateType = EnumUpdateType.REPLACE;
150-
} else if (record.getUpsertType() == UpdateType.UPDATE) {
150+
} else if (record.getUpsertType() == UpsertType.UPDATE) {
151151
updateType = EnumUpdateType.UPDATE;
152152
}
153153
Upsert upsert = Upsert.builder().uniqueColumns(record.getUpsert()).updateType(updateType).build();
@@ -171,9 +171,9 @@ protected InsertRequest getBulkInsertRequestBody(com.skyflow.vault.data.InsertRe
171171
if (request.getUpsert() != null && !request.getUpsert().isEmpty()) {
172172
if (request.getUpsertType() != null) {
173173
EnumUpdateType updateType = null;
174-
if (request.getUpsertType() == UpdateType.REPLACE) {
174+
if (request.getUpsertType() == UpsertType.REPLACE) {
175175
updateType = EnumUpdateType.REPLACE;
176-
} else if (request.getUpsertType() == UpdateType.UPDATE) {
176+
} else if (request.getUpsertType() == UpsertType.UPDATE) {
177177
updateType = EnumUpdateType.UPDATE;
178178
}
179179
Upsert upsert = Upsert.builder().uniqueColumns(request.getUpsert()).updateType(updateType).build();

v3/src/main/java/com/skyflow/enums/UpdateType.java renamed to v3/src/main/java/com/skyflow/enums/UpsertType.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
package com.skyflow.enums;
22

3-
public enum UpdateType {
3+
public enum UpsertType {
44
UPDATE("UPDATE"),
55

66
REPLACE("REPLACE");
77

88
private final String value;
99

10-
UpdateType(String value) {
10+
UpsertType(String value) {
1111
this.value = value;
1212
}
1313

v3/src/main/java/com/skyflow/utils/Utils.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,7 @@ public static List<DetokenizeRequest> createDetokenizeBatches(DetokenizeRequest
5454
// Create a sublist for the current batch
5555
List<String> batchTokens = tokens.subList(i, Math.min(i + batchSize, tokens.size()));
5656
List<TokenGroupRedactions> tokenGroupRedactions = null;
57-
if (request.getTokenGroupRedactions().isPresent() && !request.getTokenGroupRedactions().get().isEmpty()
58-
) {
57+
if (request.getTokenGroupRedactions().isPresent() && !request.getTokenGroupRedactions().get().isEmpty()){
5958
tokenGroupRedactions = request.getTokenGroupRedactions().get();
6059
}
6160
// Build a new DetokenizeRequest for the current batch

v3/src/main/java/com/skyflow/vault/data/DetokenizeResponseObject.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@
66
import java.util.Map;
77

88
public class DetokenizeResponseObject {
9+
@Expose(serialize = true)
10+
private int index;
11+
912
@Expose(serialize = true)
1013
private String token;
1114
@Expose(serialize = true)
@@ -15,9 +18,6 @@ public class DetokenizeResponseObject {
1518
@Expose(serialize = true)
1619
private String error;
1720

18-
@Expose(serialize = true)
19-
private int index;
20-
2121
@Expose(serialize = true)
2222
private Map<String, Object> metadata;
2323

v3/src/main/java/com/skyflow/vault/data/InsertRecord.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package com.skyflow.vault.data;
22

3-
import com.skyflow.enums.UpdateType;
3+
import com.skyflow.enums.UpsertType;
44

55
import java.util.List;
66
import java.util.Map;
@@ -25,7 +25,7 @@ public List<String> getUpsert() {
2525
return this.builder.upsert;
2626
}
2727

28-
public UpdateType getUpsertType() {
28+
public UpsertType getUpsertType() {
2929
return this.builder.upsertType;
3030
}
3131

@@ -34,7 +34,7 @@ public static final class InsertRecordBuilder {
3434
private String table;
3535
private Map<String, Object> data;
3636
private List<String> upsert;
37-
private UpdateType upsertType;
37+
private UpsertType upsertType;
3838

3939
public InsertRecordBuilder table(String table) {
4040
this.table = table;
@@ -51,7 +51,7 @@ public InsertRecordBuilder upsert(List<String> upsert) {
5151
return this;
5252
}
5353

54-
public InsertRecordBuilder upsertType(UpdateType upsertType) {
54+
public InsertRecordBuilder upsertType(UpsertType upsertType) {
5555
this.upsertType = upsertType;
5656
return this;
5757
}

v3/src/main/java/com/skyflow/vault/data/InsertRequest.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,8 @@
11
package com.skyflow.vault.data;
22

3-
import com.skyflow.enums.UpdateType;
3+
import com.skyflow.enums.UpsertType;
44

55
import java.util.ArrayList;
6-
import java.util.HashMap;
76
import java.util.List;
87

98
public class InsertRequest extends BaseInsertRequest {
@@ -23,7 +22,7 @@ public List<String> getUpsert() {
2322
return this.builder.upsert;
2423
}
2524

26-
public UpdateType getUpsertType() {
25+
public UpsertType getUpsertType() {
2726
return this.builder.upsertType;
2827
}
2928

@@ -34,7 +33,7 @@ public ArrayList<InsertRecord> getRecords(){
3433
public static final class InsertRequestBuilder extends BaseInsertRequestBuilder {
3534
private List<String> upsert;
3635

37-
private UpdateType upsertType;
36+
private UpsertType upsertType;
3837

3938
private ArrayList<InsertRecord> records;
4039

@@ -53,7 +52,7 @@ public InsertRequestBuilder upsert(List<String> upsert) {
5352
return this;
5453
}
5554

56-
public InsertRequestBuilder upsertType(UpdateType upsertType) {
55+
public InsertRequestBuilder upsertType(UpsertType upsertType) {
5756
this.upsertType = upsertType;
5857
return this;
5958
}

v3/src/test/java/com/skyflow/VaultClientTests.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import com.skyflow.config.Credentials;
44
import com.skyflow.config.VaultConfig;
55
import com.skyflow.enums.Env;
6-
import com.skyflow.enums.UpdateType;
6+
import com.skyflow.enums.UpsertType;
77
import com.skyflow.errors.ErrorCode;
88
import com.skyflow.errors.SkyflowException;
99
import com.skyflow.generated.rest.resources.recordservice.RecordserviceClient;
@@ -217,7 +217,7 @@ public void testUpsertAtRequestLevel() {
217217
com.skyflow.vault.data.InsertRequest.builder()
218218
.records(records)
219219
.upsert(upsertColumns)
220-
.upsertType(UpdateType.REPLACE)
220+
.upsertType(UpsertType.REPLACE)
221221
.build();
222222

223223
InsertRequest result = vaultClient.getBulkInsertRequestBody(request, vaultConfig);
@@ -231,7 +231,7 @@ public void testUpsertAtRecordLevel() {
231231
Map<String, Object> data = new HashMap<>();
232232
data.put("key", "value");
233233
List<String> upsertColumns = Arrays.asList("col2");
234-
InsertRecord record = InsertRecord.builder().data(data).upsert(upsertColumns).upsertType(UpdateType.UPDATE).build();
234+
InsertRecord record = InsertRecord.builder().data(data).upsert(upsertColumns).upsertType(UpsertType.UPDATE).build();
235235
System.out.println("record upsert: " + record.getUpsertType());
236236
ArrayList<InsertRecord> records = new ArrayList<>();
237237
records.add(record);

0 commit comments

Comments
 (0)