GitHub user ahutsunshine opened a pull request:
https://github.com/apache/incubator-griffin/pull/320
refactor job structure to remove redundant class and api
1.refactor job structure
2.remove JobSchedule class to make job structure more clear
3.remove JobDataBean to make resp
GitHub user jasonliaoxiaoge opened a pull request:
https://github.com/apache/incubator-griffin/pull/321
rename spark.url to resource manager url and log exception stack tree
rename spark.url to resource manager url and log exception stack tree
You can merge this pull request into a
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-griffin/pull/319
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/321
LGTM.
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/320
hi Yao,
there are some conflicting files, could you fix them then we will review
this patch.
Thanks,
William
---
GitHub user toyboxman opened a pull request:
https://github.com/apache/incubator-griffin/pull/322
Refactor/update cases
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/toyboxman/incubator-griffin
refactor/update-cases
Alternati
Github user toyboxman closed the pull request at:
https://github.com/apache/incubator-griffin/pull/322
---
[
https://issues.apache.org/jira/browse/GRIFFIN-114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16519898#comment-16519898
]
wanyin commented on GRIFFIN-114:
Sent the PR: https://github.com/apache/incubator-griffi
GitHub user toyboxman opened a pull request:
https://github.com/apache/incubator-griffin/pull/323
refactor test codes in service/src/test/java/org/apache/griffin/core/â¦
â¦util/PropertiesUtilTest.java
1.remove assert keyword instead of Junit Assert class
You can merge thi
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/321
rename spark to resource manager is a good proposal, but rm always remind
me the remove command, maybe we need to use full name as 'resource.manager',
that makes things clear.
Github user jasonliaoxiaoge commented on the issue:
https://github.com/apache/incubator-griffin/pull/321
Done, change to yarn url
---
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-griffin/pull/323
---
Github user ahutsunshine commented on the issue:
https://github.com/apache/incubator-griffin/pull/320
hi @guoyuepeng ï¼
I have solved conflicting files. If you have time, pls review this pr.
Thanks,
Kevin Yao
---
13 matches
Mail list logo