search

uwescience / raco

Compilation and rule-based optimization framework for relational algebra. Raco is the language, optimization, and query translation layer for the Myria project.
Other
72 stars 19 forks source link

Integrate MyriaL support for `sample()` function #542

Open senderista opened 8 years ago

senderista commented 8 years ago

Looks like the sample() operator never made it into https://github.com/uwescience/raco/pull/428. I thought that the expected plans generated by sample() were equivalent to https://github.com/uwescience/myria/blob/master/jsonQueries/radion_queries/SampleWR.json and https://github.com/uwescience/myria/blob/master/jsonQueries/radion_queries/SampleWoR.json, but those plans just seem to be samplescan() with presampling. I think we actually need a new MyriaX reservoir sampling operator to properly support this function: https://github.com/uwescience/myria/issues/859.