[GitHub] flink issue #2731: [FLINK-4934] [gelly] Triadic Census

2016-11-07 Thread greghogan
Github user greghogan commented on the issue:

https://github.com/apache/flink/pull/2731
  
@vasia the graphs have been removed.


---
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] flink issue #2731: [FLINK-4934] [gelly] Triadic Census

2016-11-06 Thread greghogan
Github user greghogan commented on the issue:

https://github.com/apache/flink/pull/2731
  
The RMat graph is used across many tests and the intent was to verify the 
larger tests against secondary frameworks. I have no problem leaving out the 
files as the command is documented in the base test class.


---
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] flink issue #2731: [FLINK-4934] [gelly] Triadic Census

2016-11-06 Thread vasia
Github user vasia commented on the issue:

https://github.com/apache/flink/pull/2731
  
Hi @greghogan,
do we really need a 12k-line csv and a 32k-line csv to test this?


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