Github user justinleet commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/256#discussion_r79158991
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/hadoop/SequenceFileIterable.java
---
@@ -0,0 +1,138 @@
+/**
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/256
CI failure seems unrelated. Probably something we need to address, but a
re-run will probably fix it for this PR.
```
Failed tests:
StellarStatisticsFunctionsTest.tes
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/256
It's troubling because that RNG is seeded and the results should be
deterministic. I wonder if the t-digest merge has some non-determinism.
Anyway, it needs to be fixed, but definitely n
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/255
---
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 featu
GitHub user cestella opened a pull request:
https://github.com/apache/incubator-metron/pull/257
METRON-426: Stellar does not support scientific notation as a literal
Stellar does not support scientific notation for expressing literal
doubles. It should.
This is exercised in
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/256
FYI: That test nondeterminism should be fixed as of METRON-426 aka #257
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/257
The issue around the `StellarScientificTest` change is that merging
t-digests is not precisely the same as adding the same data to one t-digest.
Whether this is a bug in the t-digest code
I went ahead and created a Jira ticket mirroring Casey's discussion of the
MVP. Feel free to add anything of interest there, too.
https://issues.apache.org/jira/browse/METRON-427
Justin
On Fri, Sep 16, 2016 at 9:39 AM, Justin Leet wrote:
>
> -- Forwarded message --
> From: z
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/252
merged master
---
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 wishe
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/254
Merged with master
---
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
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/252
+1
---
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 t
Github user justinleet commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/257#discussion_r79204587
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/math/stats/OnlineStatisticsProvider.java
---
@@ -43,7 +43,7 @@
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/249
+1
---
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 t
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/252
---
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 featu
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/249
---
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 featu
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/257#discussion_r79245698
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/math/stats/OnlineStatisticsProvider.java
---
@@ -43,7 +43,7 @@
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/254
+1
---
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 t
GitHub user cestella opened a pull request:
https://github.com/apache/incubator-metron/pull/258
METRON-428: Allow a kafka offset to be passed to the ParserTopology CLI
Currently the parser start utility does not support passing whether we want
to start the topology at the beginning
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/254
---
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 featu
GitHub user nickwallen opened a pull request:
https://github.com/apache/incubator-metron/pull/259
METRON-429 Profiler Missing Dependencies When Uber Jar Deployed
When the Profiler is packaged and deployed on a Metron environment, the
topology does not start because the uber jar is m
20 matches
Mail list logo