[jira] [Comment Edited] (METRON-629) Modify Stellar function BLOOM_ADD to accept a list

["Otto Fowler (JIRA)" <ji...@apache.org>]

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

Otto Fowler edited comment on METRON-629 at 11/17/17 5:50 AM:
--------------------------------------------------------------

BLOOM supports Lists as it does Maps and other objects. Supporting a List as a container for individual objects where you want:


{code:java}
BLOOM_ADD(BLOOM_INIT(),'one','two') 
{code}


to be the same logically as:


{code:java}
BLOOM_ADD(BLOOM_INIT(),['one','two'])
{code}


Will make it impossible to support Lists as Lists if you know what I mean.
I'm not sure we want to do this.

Thoughts [~cestella]?


was (Author: ottobackwards):
BLOOM supports Lists as it does Maps and other objects. Supporting a List as a container for individual objects where you want:

BLOOM_ADD(BLOOM_INIT(),'one','two') 

to be the same logically as:

BLOOM_ADD(BLOOM_INIT(),['one','two'])

Will make it impossible to support Lists as Lists if you know what I mean.
I'm not sure we want to do this.

Thoughts [~cestella]?

> Modify Stellar function BLOOM_ADD to accept a list
> --------------------------------------------------
>
>                 Key: METRON-629
>                 URL: https://issues.apache.org/jira/browse/METRON-629
>             Project: Metron
>          Issue Type: Improvement
>            Reporter: Michael Miklavcic
>
> Current functionality is BLOOM_ADD(filter, val1, val2, valn). To keep consistent with other usage in the system (for instance, the profiler) this should be changed to support a single value or list of values.
> BLOOM_ADD(filter, val)
> BLOOM_ADD(filter, [ val1, val2, valn ] )



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)