Skip to content

Commit abb53e2

Browse files
khorshuhengzhilingc
authored andcommitted
Remove feature set status check for job update requirement
1 parent 1f40995 commit abb53e2

1 file changed

Lines changed: 1 addition & 12 deletions

File tree

core/src/main/java/feast/core/job/JobUpdateTask.java

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
package feast.core.job;
1818

1919
import com.google.common.collect.Sets;
20-
import feast.core.FeatureSetProto.FeatureSetStatus;
2120
import feast.core.log.Action;
2221
import feast.core.log.AuditLogger;
2322
import feast.core.log.Resource;
@@ -103,17 +102,7 @@ public Job call() {
103102

104103
boolean requiresUpdate(Job job) {
105104
// If set of feature sets has changed
106-
if (!Sets.newHashSet(featureSets).equals(Sets.newHashSet(job.getFeatureSets()))) {
107-
return true;
108-
}
109-
110-
// If any existing feature set populated by the job has its status as pending
111-
for (FeatureSet featureSet : job.getFeatureSets()) {
112-
if (featureSet.getStatus().equals(FeatureSetStatus.STATUS_PENDING)) {
113-
return true;
114-
}
115-
}
116-
return false;
105+
return !Sets.newHashSet(featureSets).equals(Sets.newHashSet(job.getFeatureSets()));
117106
}
118107

119108
private Job createJob() {

0 commit comments

Comments
 (0)