mmiklavc commented on a change in pull request #1416: METRON-2123: Expand
Stellar JOIN to work on all Iterables
URL: https://github.com/apache/metron/pull/1416#discussion_r287169086
##########
File path:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/StringFunctions.java
##########
@@ -133,15 +131,15 @@ public Object apply(List<Object> strings) {
}
@Stellar( name="JOIN"
- , description="Joins the components in the list of strings with the
specified delimiter."
- , params = { "list - List of strings", "delim - String delimiter"}
+ , description="Joins the non-null items in the iterable as strings
with the specified delimiter. Null items are dropped."
+ , params = { "iterable - iterable (e.g. List, LinkedHashSet, etc.)
of items treated as strings", "delim - String delimiter"}
, returns = "String"
)
Review comment:
I'm sorry, I don't think I follow - line 140 just above the `apply(...)`
method already has a "@SuppressWarnings" annotation. Are you asking whether we
need to remove it, or were you suggesting adding another one somewhere?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
