[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...
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 this. Thanks @MohanDV ---
[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...
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...
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...
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...
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 [METRON-1450](https://issues.apache.org/jira/projects/METRON/issues/METRON-1450) the summary is: ``` Add documentation for splitting the indexing topology into random_access_indexing and batch_indexing ``` Based on that description, this would be the ticket to address the indexing README. If you think it's worth splitting this into two JIRAs that's fine with me, but I already had [METRON-1454](https://issues.apache.org/jira/projects/METRON/issues/METRON-1454) open to remediate the main indexing README, but I closed it because the description of [METRON-1450](https://issues.apache.org/jira/projects/METRON/issues/METRON-1450) mirrored what I was looking to accomplish. Let me know where you'd like to go from here, but regardless we should make sure that the PR title here matches the JIRA summary. ---
[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...
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...
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 longer accurate. ---
[GitHub] metron issue #936: METRON-1450:Added documentation for random access and bat...
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...
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...
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...
Github user JonZeolla commented on the issue: https://github.com/apache/metron/pull/936 Would you also mind updating the main README? ---