falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ying Zheng" <yzh...@hortonworks.com>
Subject Re: Review Request 32946: Falcon-1121: Backend Support for Free-text Entity Search
Date Thu, 16 Apr 2015 23:34:23 GMT


> On April 8, 2015, 6:04 a.m., Ajay Yadava wrote:
> > client/src/main/java/org/apache/falcon/cli/FalconCLI.java, line 101
> > <https://reviews.apache.org/r/32946/diff/1/?file=920312#file920312line101>
> >
> >     How is the new tagkey_opt different from the current tag option?
> 
> Ying Zheng wrote:
>     With the old "tag" option, the user needs to specify the full tag string, while with
new “tagkey” option, they can specify only a substring in the tag. For example, the user
can search the entity with tag "category=realestate” by tagkey “estate”.
> 
> Pallavi Rao wrote:
>     tagkey kind of implies that it is the key in the key-value pair of the tag. Can we
call it tagPattern or tagSubstring, instead?
>     
>     Also, it is a little redundant and could be confusing to the users with two parameters
to search on tags , filterByTags and this. Could we not just add support for substring search
in filterTags itself?

1. The original proposal is to use "tagkey" to stand for tag keywords. See detailed proposal:
https://issues.apache.org/jira/browse/FALCON-1095. We can call "tagkeywords", but the user
will need to type longer command on CLI. It should not be confusing as long as we document
the api well. Let me know what you think.

2. Good question. The major concern is the backward compatibility. Since the users have been
using the "tags" option for exact tag matching, it may affect the existing usages if we change
its functionality.


- Ying


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32946/#review79308
-----------------------------------------------------------


On April 8, 2015, 1:05 a.m., Ying Zheng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32946/
> -----------------------------------------------------------
> 
> (Updated April 8, 2015, 1:05 a.m.)
> 
> 
> Review request for Falcon, Ajay Yadava, Sowmya Ramesh, Seetharam Venkatesh, and Venkat
Ranganathan.
> 
> 
> Bugs: Falcon-1121
>     https://issues.apache.org/jira/browse/Falcon-1121
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> See parent JIRA Falcon-1095 for detailed descriptions: http://issues.apache.org/jira/browse/Falcon-1095
> Four changes in total:
> 1. Added API support: return feeds and processes if :entity-type is set to schedulable;
change parameter for subsequence matching from "pattern" to "nameseq"; multi-keyword matching
for entity tags with parameter "tagkey".
> 2. Added command line support.
> 3. Added unit tests.
> 4. Updated twiki.
> 
> 
> Diffs
> -----
> 
>   client/src/main/java/org/apache/falcon/cli/FalconCLI.java 958ea62 
>   client/src/main/java/org/apache/falcon/client/FalconClient.java 15691a6 
>   client/src/main/java/org/apache/falcon/entity/v0/Entity.java 7fb271d 
>   client/src/main/java/org/apache/falcon/resource/EntityList.java d43418c 
>   common/src/main/java/org/apache/falcon/entity/store/ConfigurationStore.java 1c1c325

>   docs/src/site/twiki/FalconCLI.twiki 22ffbe7 
>   docs/src/site/twiki/restapi/EntityList.twiki 5e11691 
>   prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java c0df4d6 
>   prism/src/main/java/org/apache/falcon/resource/AbstractSchedulableEntityManager.java
f7c2f61 
>   prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java
8bfc099 
>   prism/src/test/java/org/apache/falcon/resource/EntityManagerTest.java 6f75111 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java 261a9a9

>   webapp/src/test/java/org/apache/falcon/cli/FalconCLIIT.java bfad011 
>   webapp/src/test/java/org/apache/falcon/cli/FalconCLISmokeIT.java ee5534a 
>   webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseyIT.java 280253d

> 
> Diff: https://reviews.apache.org/r/32946/diff/
> 
> 
> Testing
> -------
> 
> Unit tests passed. Also tested command line and using api calls.
> 
> 
> Thanks,
> 
> Ying Zheng
> 
>


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