[GitHub] incubator-metron issue #260: METRON-433: Documentation update

2016-09-20 Thread mmiklavc
Github user mmiklavc commented on the issue:

https://github.com/apache/incubator-metron/pull/260
  
@anandsubbu @JonZeolla Thank you both for the feedback and contributions!


---
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.
---


[GitHub] incubator-metron issue #260: METRON-433: Documentation update

2016-09-20 Thread cestella
Github user cestella commented on the issue:

https://github.com/apache/incubator-metron/pull/260
  
You got it; I'll merge this guy later on today and you can submit a new PR 
that replaces the images. :)  Thanks for the contribution!


---
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.
---


[GitHub] incubator-metron issue #260: METRON-433: Documentation update

2016-09-20 Thread anandsubbu
Github user anandsubbu commented on the issue:

https://github.com/apache/incubator-metron/pull/260
  
Hi @cestella and @JonZeolla, great. I will go ahead and create similar ones 
for the remaining diagrams as well. 

Either way works fine by me, though I would be super excited to get a 
chance at my very first commit :). 


---
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.
---


[GitHub] incubator-metron issue #260: METRON-433: Documentation update

2016-09-20 Thread JonZeolla
Github user JonZeolla commented on the issue:

https://github.com/apache/incubator-metron/pull/260
  
@anandsubbu That's exactly what I had in mind.  @cestella, thoughts?  


---
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.
---


[GitHub] incubator-metron issue #260: METRON-433: Documentation update

2016-09-20 Thread anandsubbu
Github user anandsubbu commented on the issue:

https://github.com/apache/incubator-metron/pull/260
  
Hi @JonZeolla, please see if the attached sample for Enrichment is better.

![enrichment-architecture](https://cloud.githubusercontent.com/assets/20395490/18676597/c82978ba-7f73-11e6-9b56-7fecdfee6d03.png)



---
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.
---