[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-27 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/936 The PR title here doesn't match the [JIRA](https://issues.apache.org/jira/projects/METRON/issues/METRON-1450?filter=allopenissues) but as long as we account for that during merging, I'm a +1 on

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-27 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/936 +1 Thanks for correcting this @MohanDV. This looks good. Let's give @JonZeolla a chance to respond before we merge this. ---

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-27 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/936 Bump. Can anyone offer a binding +1 ? I Would like to get this in. ---

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-16 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/936 @JonZeolla Sorry I dint notice the METRON-1454 , I have updated the description for METRON-1450. I will mark my METRON-1458 as the duplicate of METRON-1454. ---

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-16 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/936 I don't see a PR for METRON-1458, did you mean you opened a [JIRA](https://issues.apache.org/jira/projects/METRON/issues/METRON-1458)? In the JIRA for

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-16 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/936 @JonZeolla it's out of the scope of this PR, This PR is intended to update the doc for rest endpoints. I have created another PR (METRON-1458) to update the main README for indexing. ---

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-16 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/936 I'm specifically referring to [this](https://github.com/MohanDV/metron/blob/4cb99e7186c408159cb41b0e5e8761fdcbfe74d2/metron-platform/metron-indexing/README.md#indexing-architecture) which is no

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-14 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/936 updated the main README for indexing ---

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-14 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/936 Yes ---

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-13 Thread MohanDV
Github user MohanDV commented on the issue: https://github.com/apache/metron/pull/936 @JonZeolla are you referring to metron-platform/metron-indexing/README.md ? ---

[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...

2018-02-13 Thread JonZeolla
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/936 Would you also mind updating the main README? ---