falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pallavi Rao" <pallavi....@inmobi.com>
Subject Re: Review Request 42064: FALCON-1662 Ensure entity can be scheduled on multiple clusters on same colo
Date Fri, 08 Jan 2016 11:00:32 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Jan. 8, 2016, 11 a.m.)

Review request for Falcon.


Made delete idempotent

Bugs: FALCON-1662

Repository: falcon-git


Some problems that were identified while testing this have been fixed:
1. OozieWorkflowEngine isActive method had an issue.
2. The JDBCStateStore was not filtering by cluster.

Additionally, noted that update was not compliant with oozie scheduler and fixed it.

Diffs (updated)

  oozie/src/main/java/org/apache/falcon/workflow/engine/OozieWorkflowEngine.java cf0e30d 
  scheduler/src/main/java/org/apache/falcon/execution/FalconExecutionService.java f45ec98

  scheduler/src/main/java/org/apache/falcon/state/store/jdbc/InstanceBean.java 5ed3ccd 
  scheduler/src/main/java/org/apache/falcon/state/store/jdbc/JDBCStateStore.java abd4119 
  scheduler/src/main/java/org/apache/falcon/workflow/engine/FalconWorkflowEngine.java c9e6da4

  scheduler/src/test/java/org/apache/falcon/state/service/store/TestJDBCStateStore.java bb8ff61

Diff: https://reviews.apache.org/r/42064/diff/


UTs updated. Manual testing done.


Pallavi Rao

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message