[ 
https://issues.apache.org/jira/browse/SHINDIG-560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12665271#action_12665271
 ] 

Ian Boston commented on SHINDIG-560:
------------------------------------

This is the last blocker, I would like tor try and get it in.

A question, AFAICT, /person/@supportedFields is not a regex, and its always 
going to be at the end of the URL.

Is there are reason that a Pattern is used rather than just saying endsWith ?
It may sound minor, but I believe that a regex pattern match is generally 
slower than very simple String comparison.

Other than that, I am happy to apply the patch.. provided it still will apply 
to both trunk and branch.

> Add support for @supportedFields
> --------------------------------
>
>                 Key: SHINDIG-560
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-560
>             Project: Shindig
>          Issue Type: New Feature
>          Components: RESTful API (Java)
>            Reporter: Cassie Doll
>            Priority: Blocker
>         Attachments: ContainerConf.java, fix-560-bug.patch, 
> JsonContainerConf.java, shindigpatch-560.patch
>
>
> We need to add support for the @supportedFields query on each handler. (ie 
> people/@supportedFields, activities/@supportedFields) 
> Maybe a guice binding to a set of strings or something similar would make 
> this work. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to