[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654329#comment-16654329
]
ASF GitHub Bot commented on METRON-1816:
Github user merrimanr commented on the
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/1233
The existing Stellar date functions use SimpleDateFormat and already have a
caching layer built in so I decided to reuse what was already there. Happy to
switch to DateTimeFormatter here or in a
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654274#comment-16654274
]
ASF GitHub Bot commented on METRON-1816:
Github user simonellistonball commented
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1233
Right, I hear that! I also really like the idea of a follow on function
that returns the default formats as a great solution to this problem. Should we
do that?
---
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654268#comment-16654268
]
ASF GitHub Bot commented on METRON-1816:
Github user ottobackwards commented on
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1233
This comes to mind for me, because using Nifi processors that take date
formats and in other places I'm sick of cutting and pasting complex ISO formats
into configurations when there are named
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654263#comment-16654263
]
ASF GitHub Bot commented on METRON-1816:
Github user simonellistonball commented
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1233
I would worry about having to parse the strong argument to determine
whether to use a constant or custom formatter, but I guess my concern is
limited because weâre caching the formatter
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654241#comment-16654241
]
ASF GitHub Bot commented on METRON-1816:
Github user ottobackwards commented on
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1233
I'm not sure what you mean? Can you elaborate?
Also, I'm actually thinking of using DateFormatter and the new java time
api's:
https://docs.oracle.com/javase/8/docs/api/java/time
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1233
@ottobackwards wouldn't ISO constant support require us to do a check on
the string before passing to the DateFormat instance, which on something that
is likely to be run A LOT (i.e. multi
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654183#comment-16654183
]
ASF GitHub Bot commented on METRON-1816:
Github user simonellistonball commented
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654078#comment-16654078
]
ASF GitHub Bot commented on METRON-1816:
Github user ottobackwards commented on
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654077#comment-16654077
]
ASF GitHub Bot commented on METRON-1816:
Github user ottobackwards commented on
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/1233#discussion_r226064248
--- Diff:
metron-stellar/stellar-common/src/test/java/org/apache/metron/stellar/dsl/functions/DateFunctionsTest.java
---
@@ -225,4 +226,36 @@ public
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/1233#discussion_r226064136
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/DateFunctions.java
---
@@ -109,6 +110,13 @@ public stat
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16654075#comment-16654075
]
ASF GitHub Bot commented on METRON-1816:
Github user ottobackwards commented on
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1233
we should support the ISO date format constants so that users don't have to
copy those formats.
---
[
https://issues.apache.org/jira/browse/METRON-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16653902#comment-16653902
]
ASF GitHub Bot commented on METRON-1816:
Github user simonellistonball commented
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/1233
Looks like a very useful gap filler to me. +1 (non-binding) by inspection
from me for sure.
---
[
https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16653752#comment-16653752
]
ASF GitHub Bot commented on METRON-1681:
Github user mmiklavc commented on the i
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/1213
lgtm, +1 by inspection. Thanks for the collaborative effort on this
@merrimanr and @ottobackwards!
---
[
https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16653745#comment-16653745
]
ASF GitHub Bot commented on METRON-1681:
Github user merrimanr commented on the
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/1213
The latest commit adds the Deprecated annotations to parse and
parseOptional and removes MultilineMessageParser. I also updated the
`MessageParserTest` with more tests to ensure we're handling th
[
https://issues.apache.org/jira/browse/METRON-1830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16653682#comment-16653682
]
ASF GitHub Bot commented on METRON-1830:
GitHub user sardell opened a pull reque
GitHub user sardell opened a pull request:
https://github.com/apache/metron/pull/1240
METRON-1830: Re-implement Alerts dialog box without jQuery
## Contributor Comments
[Link to METRON-1830 on ASF
JIRA](https://issues.apache.org/jira/browse/METRON-1830)
Previously, the
[
https://issues.apache.org/jira/browse/METRON-1830?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shane Ardell reassigned METRON-1830:
Assignee: Shane Ardell
> Re-implement Alerts dialog box without jQuery
>
Shane Ardell created METRON-1830:
Summary: Re-implement Alerts dialog box without jQuery
Key: METRON-1830
URL: https://issues.apache.org/jira/browse/METRON-1830
Project: Metron
Issue Type: Bu
[
https://issues.apache.org/jira/browse/METRON-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16653370#comment-16653370
]
ASF GitHub Bot commented on METRON-1681:
Github user ottobackwards commented on
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1213
That is fine
---
[
https://issues.apache.org/jira/browse/METRON-1803?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16653171#comment-16653171
]
ASF GitHub Bot commented on METRON-1803:
Github user tiborm commented on the iss
Github user tiborm commented on the issue:
https://github.com/apache/metron/pull/1226
@mmiklavc Yes, that's the main file. Cypress.conf.json contains 4 line of
configs and package.json 3 lines of changes required by the integration.
Anything else is test response data.
---
32 matches
Mail list logo