falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Balu Vellanki" <bvella...@hortonworks.com>
Subject Review Request 40121: Falcon-1372 : Retention does not work in corner cases
Date Tue, 10 Nov 2015 01:23:43 GMT

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

Review request for Falcon, Ajay Yadava, Sowmya Ramesh, and Venkat Ranganathan.

Bugs: Falcon-1372

Repository: falcon-git


The bug is caused due to the way java.util.Calendar handles DAY_OF_MONTH. In FeedHelper, the
getDate(...) method sets DAY_OF_MONTH to 0 for date patterns like ${YEAR}/${MONTH}.
This causes the month to be setback by 1. So "2015/11" will be treated as Oct 31st, 2015.
Hence this instance will be evicted. For a date like "2015/11" , Calender should be set to
cal.set(2015, 10, 1, 0, 0, 0); The MONTH should be "value - 1" and The DAY_OF_MONTH begins
with a 1 instead of 0.


  common/src/main/java/org/apache/falcon/entity/FeedHelper.java 8c55e41 
  common/src/main/java/org/apache/falcon/entity/common/FeedDataPath.java afe913d 
  common/src/test/java/org/apache/falcon/entity/FeedHelperTest.java 10dac49 

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


Testing done end2end, added few more test cases to verify getDate() method in FeedHelper works
as expected.


Balu Vellanki

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