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


Schema validation looks great :)


datafu-pig/src/main/java/datafu/pig/hash/lsh/CosineDistanceHash.java
<https://reviews.apache.org/r/20608/#comment76431>

    I didn't mean use a fixed seed :)  If no seed is provided then it should 
effectively be a random seed. 



datafu-pig/src/main/java/datafu/pig/hash/lsh/LSHFunc.java
<https://reviews.apache.org/r/20608/#comment76432>

    perfect


- Matthew Hayes


On May 9, 2014, 1:04 a.m., Casey Stella wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20608/
> -----------------------------------------------------------
> 
> (Updated May 9, 2014, 1:04 a.m.)
> 
> 
> Review request for DataFu.
> 
> 
> Repository: datafu
> 
> 
> Description
> -------
> 
> From DATAFU-37: Create a set of UDFs to implement Locality Sensitive Hashing 
> in support of finding k-near neighbors. Initially, hashes associated with L1, 
> L2 and Cosine similarity should be supported.
> 
> 
> Diffs
> -----
> 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/CosineDistanceHash.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/L1PStableHash.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/L2PStableHash.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/LSHFamily.java PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/LSHFunc.java PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/RepeatingLSH.java PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/cosine/HyperplaneLSH.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/cosine/package-info.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/interfaces/LSH.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/interfaces/LSHCreator.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/interfaces/Sampler.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/interfaces/package-info.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/metric/Cosine.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/metric/L1.java PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/metric/L2.java PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/metric/MetricUDF.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/metric/package-info.java 
> PRE-CREATION 
>   
> datafu-pig/src/main/java/datafu/pig/hash/lsh/p_stable/AbstractStableDistributionFunction.java
>  PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/p_stable/L1LSH.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/p_stable/L2LSH.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/p_stable/package-info.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/package-info.java PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/util/DataTypeUtil.java 
> PRE-CREATION 
>   datafu-pig/src/main/java/datafu/pig/hash/lsh/util/package-info.java 
> PRE-CREATION 
>   datafu-pig/src/test/java/datafu/test/pig/hash/lsh/LSHPigTest.java 
> PRE-CREATION 
>   datafu-pig/src/test/java/datafu/test/pig/hash/lsh/LSHTest.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/20608/diff/
> 
> 
> Testing
> -------
> 
> 2 unit tests.  One pigunit for the UDFs and one regular JUnit test to test 
> functionality.
> 
> 
> Thanks,
> 
> Casey Stella
> 
>

Reply via email to