[GitHub] wonook edited a comment on issue #103: [NEMO-11] Generalize Equality of Int Predicates for Loops

2018-08-16 Thread GitBox
wonook edited a comment on issue #103: [NEMO-11] Generalize Equality of Int Predicates for Loops URL: https://github.com/apache/incubator-nemo/pull/103#issuecomment-413768100 Hi @arunlakshman, sorry for my late review, I'm currently on holiday for the moment. Refactoring the code into a

[GitHub] wonook commented on issue #103: [NEMO-11] Generalize Equality of Int Predicates for Loops

2018-08-16 Thread GitBox
wonook commented on issue #103: [NEMO-11] Generalize Equality of Int Predicates for Loops URL: https://github.com/apache/incubator-nemo/pull/103#issuecomment-413768100 Hi @arunlakshman, sorry for my late review, I'm currently on holiday for the moment. Refactoring the code into a util p

[GitHub] johnyangk closed pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk closed pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of proven

Nemo JIRA does not allow anonymous access

2018-08-16 Thread Jangho Seo
To Nemo PMC, Recently I got noticed that Nemo JIRA[1] requires ASF JIRA account for browsing issues. That's not the case with with other ASF projects, such as Beam[2], AVRO[3], etc. Whether it was intended or not, I think we'd better change this policy, because I believe publicly accessible issue

[GitHub] johnyangk commented on issue #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on issue #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#issuecomment-413750417 Sounds good. I'll merge once the tests pass. Thi

[GitHub] seojangho commented on issue #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on issue #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#issuecomment-413749675 @johnyangk Thanks for the review. Please check the updates. -

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210797773 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

Re: Formatter for Eclipse IDE

2018-08-16 Thread Jangho Seo
Hi Nemo devs, I just filed a PR to create formatter.xml. Please check https://github.com/apache/incubator-nemo/pull/106 for the details. Comments/patches from Eclipse users (including Arun, of course!) are more than welcome. By the way, welcome to the community, Arun! Regards, Jangho On 8/17/1

[GitHub] seojangho opened a new pull request #106: [NEMO-142] Add .editorconfig and formatter.xml

2018-08-16 Thread GitBox
seojangho opened a new pull request #106: [NEMO-142] Add .editorconfig and formatter.xml URL: https://github.com/apache/incubator-nemo/pull/106 JIRA: [NEMO-142: Add .editorconfig and formatter.xml](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-142) **Major changes:** -

[GitHub] sanha closed pull request #105: [NEMO-184] Upgrade to Beam 2.6.0

2018-08-16 Thread GitBox
sanha closed pull request #105: [NEMO-184] Upgrade to Beam 2.6.0 URL: https://github.com/apache/incubator-nemo/pull/105 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull req

[GitHub] johnyangk opened a new pull request #105: [NEMO-184] Upgrade to Beam 2.6.0

2018-08-16 Thread GitBox
johnyangk opened a new pull request #105: [NEMO-184] Upgrade to Beam 2.6.0 URL: https://github.com/apache/incubator-nemo/pull/105 JIRA: [NEMO-184: Upgrade to Beam 2.6.0](https://issues.apache.org/jira/projects/NEMO/issues/NEMO-184) **Major changes:** - Upgrade to Beam 2.6.0

[GitHub] johnyangk closed pull request #100: [NEMO-178] Compile-time task cloning

2018-08-16 Thread GitBox
johnyangk closed pull request #100: [NEMO-178] Compile-time task cloning URL: https://github.com/apache/incubator-nemo/pull/100 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] johnyangk commented on issue #100: [NEMO-178] Compile-time task cloning

2018-08-16 Thread GitBox
johnyangk commented on issue #100: [NEMO-178] Compile-time task cloning URL: https://github.com/apache/incubator-nemo/pull/100#issuecomment-413713872 I'll refile the PR using an alternative design that reuses existing task attempts, rather than introducing a separate concept of clone offset

Re: Formatter for Eclipse IDE

2018-08-16 Thread John Yang
Hi Arun, AFAIK, Intellij is mostly used for development. I have very little experience with Eclipse, but would exporting the project from Intellij to Eclipse help? A quick google search with "export intellij into eclipse" shows some options for that, but I do not know whether they are related to

Re: Formatter for Eclipse IDE

2018-08-16 Thread Byung-Gon Chun
Hi Arun, Welcome to the community! I've been using Intellij. Anyone who can share formatter.xml for Eclipse? Thanks! -Gon On Fri, Aug 17, 2018 at 1:43 AM, Arun Lakshman wrote: > Hi All, > >I thank everyone for the warm welcome really looking forward to be a > part of the community. > >

Formatter for Eclipse IDE

2018-08-16 Thread Arun Lakshman
Hi All, I thank everyone for the warm welcome really looking forward to be a part of the community. I am using eclipse IDE for Nemo development. I am looking for formatter.xml for the Eclipse IDE, it will be highly helpful if any of the folks can share the formatter for the Nemo project. I

[GitHub] johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210535793 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210535793 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210525002 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210524686 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210523963 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210522713 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210512788 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210507204 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210505301 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210505301 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210506710 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
seojangho commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210505301 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210491889 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210500940 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210493130 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/

[GitHub] johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG

2018-08-16 Thread GitBox
johnyangk commented on a change in pull request #104: [NEMO-183] DAG-centric translation from Beam pipeline to IR DAG URL: https://github.com/apache/incubator-nemo/pull/104#discussion_r210491619 ## File path: compiler/frontend/beam/src/main/java/edu/snu/nemo/compiler/frontend/beam/