[jira] [Updated] (S2GRAPH-20) Bug in Query Option "Sample"

["Hwansung Yu (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/S2GRAPH-20?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Hwansung Yu updated S2GRAPH-20:
-------------------------------
       Assignee: Hyunsung Jo  (was: Daewon Jeong)
       Priority: Critical  (was: Major)
       Reporter: Hwansung Yu  (was: Hyunsung Jo)
    Description: 
Current implementation of {{randomInt()}} in {{AsynchbaseQueryBuiler.scala}} results in an infinite loop if {{range}} parameter is 0.
Please add a condition to handle this edge case.

  was:
*Problem*

I've noticed that S2Graph clients quite often shuffle the query results before serving them to users in order to give some randomness to user experience.
An option to randomly sample a set of queried edges will result in a much simpler client code.
For example, lets say a client is running an AB test on S2Graph items with A) a sorted bucket and B) a random bucket.
As is, she will have to identify the random bucket id B and mix up the result.
With the suggested feature, both buckets can be handled uniformly.


*Idea*

Add a step-level integer-type parameter "sample" that will tell S2Graph to randomly sample N edges from the result set of the corresponding step.

     Issue Type: Bug  (was: New Feature)
        Summary: Bug in Query Option "Sample"  (was: Add Query Option "Sample" to Provide Randomness to a Query Result)

> Bug in Query Option "Sample"
> ----------------------------
>
>                 Key: S2GRAPH-20
>                 URL: https://issues.apache.org/jira/browse/S2GRAPH-20
>             Project: S2Graph
>          Issue Type: Bug
>            Reporter: Hwansung Yu
>            Assignee: Hyunsung Jo
>            Priority: Critical
>
> Current implementation of {{randomInt()}} in {{AsynchbaseQueryBuiler.scala}} results in an infinite loop if {{range}} parameter is 0.
> Please add a condition to handle this edge case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)