Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/516#discussion_r110992116
--- Diff: metron-platform/metron-common/README.md ---
@@ -167,6 +172,14 @@ The `!=` operator is the negation of the above.
| [ `WEEK_OF_YEAR`](#week_of_year)
|
| [ `YEAR`](#year)
|
+### `APPEND_IF_MISSING`
+ * Description: Appends the suffix to the end of the string if the string
does not already end with any of the suffixes.
+ * Input:
+ * string - The string to be appended.
+ * suffix - The string suffix to append to the end of the string.
+ * suffixes - Optional - Additional string suffixes that are valid
terminators.
--- End diff --
Hi @mattf-horton , I was trying to following the same wording as the
[apache commons
documentation](https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html)
for PrependIfMissing, which uses the plural form.
Let me know if it would be better if I changed the text as follows:
`* suffix - The string suffix to append to the end of the string.`
`* additionalsuffix - Optional - An additional string suffix that is a
valid terminator.`
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
