[GitHub] incubator-griffin pull request #436: always use apache griffin together as o...

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/436#discussion_r224487868 --- Diff: griffin-doc/dev/dev-env-build.md --- @@ -85,7 +85,7 @@ mvn clean install ``` ### For service module and ui module

[GitHub] incubator-griffin pull request #435: [GRIFFIN-206] fix job timezone when cre...

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/435#discussion_r224482419 --- Diff: ui/angular/src/app/job/create-job/batch/batch.component.ts --- @@ -134,8 +134,8 @@ export class BatchComponent implements OnInit

[GitHub] incubator-griffin pull request #435: [GRIFFIN-206] fix job timezone when cre...

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/435#discussion_r224461843 --- Diff: service/src/test/java/org/apache/griffin/core/util/TimeUtilTest.java --- @@ -110,4 +111,36 @@ public void

[GitHub] incubator-griffin pull request #435: [GRIFFIN-206] fix job timezone when cre...

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/435#discussion_r224461713 --- Diff: service/src/test/java/org/apache/griffin/core/util/TimeUtilTest.java --- @@ -110,4 +111,36 @@ public void

[GitHub] incubator-griffin pull request #435: [GRIFFIN-206] fix job timezone when cre...

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/435#discussion_r224459867 --- Diff: service/src/test/java/org/apache/griffin/core/util/TimeUtilTest.java --- @@ -110,4 +111,36 @@ public void

[GitHub] incubator-griffin pull request #435: [GRIFFIN-206] fix job timezone when cre...

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/435#discussion_r224481345 --- Diff: ui/angular/src/app/measure/create-measure/configuration/configuration.component.ts --- @@ -56,47 +56,47 @@ export class

[GitHub] incubator-griffin pull request #435: [GRIFFIN-206] fix job timezone when cre...

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/435#discussion_r224479994 --- Diff: ui/angular/src/app/job/create-job/batch/batch.component.ts --- @@ -134,8 +134,8 @@ export class BatchComponent implements OnInit

[GitHub] incubator-griffin pull request #434: Griffin-205: accuracy matched fraction

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/434#discussion_r224438517 --- Diff: measure/src/test/scala/org/apache/griffin/measure/transformations/AccuracyTransformationsIntegrationTest.scala --- @@ -0,0 +1,169

[GitHub] incubator-griffin pull request #434: Griffin-205: accuracy matched fraction

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/434#discussion_r224432819 --- Diff: measure/src/main/scala/org/apache/griffin/measure/configuration/dqdefinition/DQConfig.scala --- @@ -148,12 +148,12 @@ case class

[GitHub] incubator-griffin pull request #434: Griffin-205: accuracy matched fraction

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/434#discussion_r224431040 --- Diff: measure/src/main/scala/org/apache/griffin/measure/configuration/dqdefinition/DQConfig.scala --- @@ -74,9 +74,9 @@ case class

[GitHub] incubator-griffin pull request #434: Griffin-205: accuracy matched fraction

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/434#discussion_r224425167 --- Diff: measure/src/main/scala/org/apache/griffin/measure/step/builder/dsl/transform/AccuracyExpr2DQSteps.scala --- @@ -125,14 +126,26

[GitHub] incubator-griffin pull request #434: Griffin-205: accuracy matched fraction

2018-10-11 Thread ashutakGG
Github user ashutakGG commented on a diff in the pull request: https://github.com/apache/incubator-griffin/pull/434#discussion_r224424475 --- Diff: measure/src/main/scala/org/apache/griffin/measure/step/builder/dsl/transform/AccuracyExpr2DQSteps.scala --- @@ -125,14 +126,26

[GitHub] incubator-griffin pull request #434: Griffin-205: accuracy matched fraction

2018-10-10 Thread ashutakGG
GitHub user ashutakGG opened a pull request: https://github.com/apache/incubator-griffin/pull/434 Griffin-205: accuracy matched fraction https://issues.apache.org/jira/browse/GRIFFIN-205 This pull request covers only batch dq type. We need to decide is it worth to add