falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ajay Yadava" <ajayn...@gmail.com>
Subject Re: Review Request 29050: Add path matching ability to the radix tree
Date Fri, 26 Dec 2014 01:39:07 GMT

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

(Updated Dec. 26, 2014, 1:39 a.m.)

Review request for Falcon and Srikanth Sundarrajan.


Implemented code review comments.

Bugs: FALCON-823

Repository: falcon-git


We need two types of iterations on the Radix Tree. One to find the normal string match with
the key and another one to do a regex iteration so that we can give feed's instance path and
match with feed's template. This patch adds support for latter.

Diffs (updated)

  common/src/main/java/org/apache/falcon/entity/common/FeedDataPath.java 39e636b 
  common/src/main/java/org/apache/falcon/entity/store/FeedPathStore.java 97d21c1 
  common/src/main/java/org/apache/falcon/util/FalconRadixUtils.java PRE-CREATION 
  common/src/main/java/org/apache/falcon/util/RadixNode.java 564df8e 
  common/src/main/java/org/apache/falcon/util/RadixTree.java 6dbe160 
  common/src/test/java/org/apache/falcon/entity/FeedDataPathTest.java PRE-CREATION 
  common/src/test/java/org/apache/falcon/util/RadixNodeTest.java 4f63806 
  common/src/test/java/org/apache/falcon/util/RadixTreeTest.java 28589ed 

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


UT have been augmented for the new feature.


Ajay Yadava

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