Skip to content
This repository was archived by the owner on Nov 22, 2020. It is now read-only.

Commit b1785a7

Browse files
committed
Add optional run ID to job meta
Signed-off-by: wslulciuc <[email protected]>
1 parent 2026237 commit b1785a7

File tree

3 files changed

+17
-3
lines changed

3 files changed

+17
-3
lines changed

src/main/java/marquez/client/models/Job.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ public Job(
5050
final Map<String, String> context,
5151
final String description,
5252
@Nullable final Run latestRun) {
53-
super(type, inputs, outputs, location, context, description);
53+
super(type, inputs, outputs, location, context, description, null);
5454
this.id = id;
5555
this.name = name;
5656
this.createdAt = createdAt;

src/main/java/marquez/client/models/JobMeta.java

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,20 +36,23 @@ public class JobMeta {
3636
@Nullable private final URL location;
3737
@Getter private final Map<String, String> context;
3838
@Nullable String description;
39+
@Nullable String runId;
3940

4041
public JobMeta(
4142
@NonNull final JobType type,
4243
@NonNull final Set<DatasetId> inputs,
4344
@NonNull final Set<DatasetId> outputs,
4445
@Nullable final URL location,
4546
@Nullable final Map<String, String> context,
46-
@Nullable final String description) {
47+
@Nullable final String description,
48+
@Nullable String runId) {
4749
this.type = type;
4850
this.inputs = inputs;
4951
this.outputs = outputs;
5052
this.location = location;
5153
this.context = (context == null) ? ImmutableMap.of() : ImmutableMap.copyOf(context);
5254
this.description = description;
55+
this.runId = runId;
5356
}
5457

5558
public Optional<URL> getLocation() {
@@ -60,6 +63,10 @@ public Optional<String> getDescription() {
6063
return Optional.ofNullable(description);
6164
}
6265

66+
public Optional<String> getRunId() {
67+
return Optional.ofNullable(runId);
68+
}
69+
6370
public String toJson() {
6471
return Utils.toJson(this);
6572
}
@@ -75,6 +82,7 @@ public static final class Builder {
7582
@Nullable private URL location;
7683
@Nullable private String description;
7784
@Nullable Map<String, String> context;
85+
@Nullable String runId;
7886

7987
private Builder() {
8088
this.inputs = ImmutableSet.of();
@@ -137,8 +145,13 @@ public Builder description(@Nullable String description) {
137145
return this;
138146
}
139147

148+
public Builder runId(@Nullable String runId) {
149+
this.runId = runId;
150+
return this;
151+
}
152+
140153
public JobMeta build() {
141-
return new JobMeta(type, inputs, outputs, location, context, description);
154+
return new JobMeta(type, inputs, outputs, location, context, description, runId);
142155
}
143156
}
144157
}

src/test/java/marquez/client/models/JsonGenerator.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@ public static String newJsonFor(final JobMeta meta) {
190190
obj.put("location", meta.getLocation().map(URL::toString).orElse(null));
191191
obj.set("context", context);
192192
obj.put("description", meta.getDescription().orElse(null));
193+
obj.put("runId", meta.getRunId().orElse(null));
193194

194195
return obj.toString();
195196
}

0 commit comments

Comments
 (0)