[ 
https://issues.apache.org/jira/browse/HIVE-4053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13591449#comment-13591449
 ] 

Mark Grover commented on HIVE-4053:
-----------------------------------

Krishna, thanks for doing this. I don't have a whole lot of insight into these 
particular algorithms but do they always take the same parameters? What's the 
possibility of a new phonetic algorithm using a different set or number of 
parameters?

If these functions always take same parameters, it may make sense to do (2).
However, if not, (1) would be a good idea. Of course, you can still refactor 
the code and share amongst all different UDFs even when they are separate.

To post a review on reviewboard, go to reviews.apache.org. Generate a diff file 
of your changes on top of hive trunk (using svn diff or git diff) and upload 
that diff (use "hive" repository when using svn diff output and hive-git 
repository when using git diff output).

Please let me know if you have any further questions.
                
> Add support for phonetic algorithms in Hive
> -------------------------------------------
>
>                 Key: HIVE-4053
>                 URL: https://issues.apache.org/jira/browse/HIVE-4053
>             Project: Hive
>          Issue Type: New Feature
>          Components: UDF
>    Affects Versions: 0.10.0
>            Reporter: Krishna
>              Labels: patch
>             Fix For: 0.10.0
>
>         Attachments: FunctionRegistry.java, GenericUDFRefinedSoundex.java, 
> HIVE-4053.1.patch.txt
>
>
> Following phonetic algorithms should be considered, which are very useful in 
> search:
> Soundex: http://en.wikipedia.org/wiki/Soundex
> Refined Soundex: Refer to the comment on 22/Feb/13 23:51
> Daitch–Mokotoff Soundex: 
> http://en.wikipedia.org/wiki/Daitch%E2%80%93Mokotoff_Soundex
> Metaphone and Double Metaphone: http://en.wikipedia.org/wiki/Metaphone
> New York State Identification and Intelligence System (NYSIIS): 
> http://en.wikipedia.org/wiki/New_York_State_Identification_and_Intelligence_System
> Caverphone: http://en.wikipedia.org/wiki/Caverphone

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to