Skip to content

Commit

Permalink
Fix yaml specification on feast entity showing 'jsonPath' and 'jsonPa…
Browse files Browse the repository at this point in the history
…thConfig' field instead of only 'jsonPathConfig' (#205)

Co-authored-by: Tio Pramayudi <[email protected]>
  • Loading branch information
tiopramayudi and tiopramayudi authored Jan 5, 2022
1 parent 329ce79 commit 31357c8
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions ui/src/services/transformer/TransformerConfig.js
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,7 @@ export class Pipeline {
} else if (entity.jsonPath) {
entity["fieldType"] = "JSONPath";
entity["field"] = { jsonPath: entity.jsonPath };
delete entity["jsonPath"]; // Delete this since all jsonPath will be converted to jsonPathConfig
} else if (entity.jsonPathConfig) {
entity["fieldType"] = "JSONPath";
entity["field"] = entity.jsonPathConfig;
Expand Down Expand Up @@ -234,6 +235,8 @@ export class Pipeline {
if (variable.jsonPath !== undefined && variable.jsonPath !== "") {
variable["type"] = "jsonpath";
variable["value"] = { jsonPath: variable.jsonPath };
variable["jsonPathConfig"] = { jsonPath: variable.jsonPath };
delete variable["jsonPath"]; // Delete this since all jsonPath will be converted to jsonPathConfig
} else if (variable.jsonPathConfig !== undefined) {
variable["type"] = "jsonpath";
variable["value"] = variable.jsonPathConfig;
Expand Down

0 comments on commit 31357c8

Please sign in to comment.