GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/100
20170705fixlicenseissue
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
20170705fixlicenseissue
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/101
20170705 update gitignore
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
20170705_update_gitignore
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/102
20170705 fix animate version 164
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/108
add ci
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin enableCI
Alternatively you can review and
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/109
20170720 enable ci
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin 20170720_enableCI
Alternatively
Github user guoyuepeng closed the pull request at:
https://github.com/apache/incubator-griffin/pull/109
---
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
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/110
20170720 enable ci 2
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
20170720_enableCI_2
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/111
20170720 enable ci 3
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
20170720_enableCI_3
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/112
20170720 reset test log
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
20170720_reset_test_log
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/113
20170720 remove test properties
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/121#discussion_r135491329
--- Diff: service/src/test/resources/quartz-test.sql ---
@@ -1,3 +1,22 @@
+/*
--- End diff --
comments with sql.
---
If your
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/131
20171011 upgrdate angular2 build
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/150
20171030 merge conflict fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/154
20171030 ignore files
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
20171030_ignore_files
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/155
20171030 remove generated readme
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/159
20171101 update parent artifact
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/263
Hi Lee,
Thanks for making our document great.
William
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/276
to download the sample file, where is the specification for the sample?
say we only download the head 20 records?
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/292
remove redundant keywords
remote final keywords from case class.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/293
2018 06 11 remove reduntant finals
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/295
rename field name
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
2018_06_11_refactor_measure
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/296
2018 06 11 refactor measure
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/298
add config reader test spec
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
2018_06_13_testcases
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/302
refactor persist to sink
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/304
2018 06 14 refactor
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
2018_06_14_refactor
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/309
hi YAO,
We want to use h2 in dev profile(application-dev.properties),
but for prod profile, we need to application-prod.properties untouched.
Could you change it
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/309
seems good to me, I will merge it.
Thanks,
William
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/316
hi
for compatible, we need scala-maven-plugin as 3.3.1
Thanks,
William
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/318
This closes #125
at the same time, we will close some obsolete PRs.
This closes #125
This closes #241
THis closes #242
This closes #279
You can merge this pull
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 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 thi
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/320
OK, we will review your patch.
Thanks Kevin.
William
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/321
hi Jason,
Thanks, that makes sense.
William
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/328
2018 06 28 fix job controller
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/330
hi Shreyansh-sangwan,
Thanks for your updates, but the endpoint could be any port, if you prefer
docker you can use 38080.
Thanks,
William
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/331
hi Jin,
Could you check why build failed?
Tests in error:
JobInstanceBeanRepoTest.testFindByActiveState » IllegalState Failed to
load Ap
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/333
LGTM
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/334
I agree 's' Interpolator is unnecessary for these two cases.
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/332
hi @toyboxman
Yes, we might need to new patch for all '\n'
Thanks,
William
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/343
ok.
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/357
2018 07 18 update docker doc
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
Github user guoyuepeng closed the pull request at:
https://github.com/apache/incubator-griffin/pull/357
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/358
update docker doc
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
2018_07_18_docker_doc
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/360
2018 07 19 doc update 1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
2018_07_19_doc_update_1
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/361
2018 07 19 doc update 2
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
2018_07_19_doc_update_2
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/362
2018 07 19 doc update 2
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
2018_07_19_doc_update_2
Github user guoyuepeng closed the pull request at:
https://github.com/apache/incubator-griffin/pull/362
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/363
This closes #330, This closes #286
This closes #330, This closes #286
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/373
Update batch measure config
redesign batch config interfce
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator
Github user guoyuepeng closed the pull request at:
https://github.com/apache/incubator-griffin/pull/373
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/376
update dq job spec
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
update_griffin_job_spec
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/384
@toyboxman
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/382
reviewed.
We need to update service and ui , and verify on dev branch first.
Thanks,
William
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/392
Hi @SparshSinghalHM
Thanks for your PR, but we cannot use mysql in our primary pom.xml directly
since its license not compatible with apache community.
so if you
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/392
@SparshSinghalHM
We are updating for our documents.
Thanks,
William
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/394
hi Grant,
Could you rebase your code before submit PR?
Thanks,
William
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/388
@toyboxman
hi Jin,
There are some conflicts, could you try to sync it and submit the PR again?
Thanks,
William
---
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/396#discussion_r210854573
--- Diff:
measure/src/main/scala/org/apache/griffin/measure/step/builder/dsl/expr/TreeNode.scala
---
@@ -18,28 +18,42 @@ under the License
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/396#discussion_r210855056
--- Diff:
measure/src/main/scala/org/apache/griffin/measure/step/builder/dsl/expr/TreeNode.scala
---
@@ -18,28 +18,42 @@ under the License
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/398
make ui style consistent
make ui module code style consistent
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/404
fix license header
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
fix_service_header
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/405
update gpg keys
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin update_gpg_keys
Alternatively you
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/422#discussion_r219671878
--- Diff: ui/angular/src/app/sidebar/sidebar.component.html ---
@@ -38,15 +38,15
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/421#discussion_r219672109
--- Diff:
service/src/main/java/org/apache/griffin/core/util/YarnNetUtil.java ---
@@ -56,6 +62,14 @@ public static boolean update(String url
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/424
Fiximport
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin fiximport
Alternatively you can review
Github user guoyuepeng closed the pull request at:
https://github.com/apache/incubator-griffin/pull/424
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/425
Fix case clauses
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin fix_case_clauses
Alternatively you
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/420
I don't know either.
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/426
ok
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/429
so that means we don't need to wait 30 minutes?
@toyboxman @chemikadze
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/429
ok, I will keep both 30m wait time and git quiet option.
will verify and then might reopen it later.
---
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/430
remove duplicated cache settings
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin-site/pull/10
update Nikolay as committer from contributor
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
GitHub user guoyuepeng opened a pull request:
https://github.com/apache/incubator-griffin/pull/436
always use apache griffin together as our mark
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/guoyuepeng/incubator-griffin
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/434#discussion_r224486025
--- Diff:
measure/src/main/scala/org/apache/griffin/measure/configuration/dqdefinition/DQConfig.scala
---
@@ -74,9 +74,9 @@ case class
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/435#discussion_r224947629
--- Diff: service/src/main/java/org/apache/griffin/core/util/TimeUtil.java
---
@@ -148,4 +149,11 @@ private static String refreshEscapeHashTag
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/435#discussion_r224948342
--- Diff: ui/angular/src/app/job/create-job/batch/batch.component.ts ---
@@ -134,8 +134,8 @@ export class BatchComponent implements OnInit
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/435#discussion_r224955984
--- Diff: service/src/main/java/org/apache/griffin/core/util/TimeUtil.java
---
@@ -148,4 +149,11 @@ private static String refreshEscapeHashTag
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/435#discussion_r224956283
--- Diff: ui/angular/src/app/job/create-job/batch/batch.component.ts ---
@@ -134,8 +134,8 @@ export class BatchComponent implements OnInit
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/432#discussion_r224948476
--- Diff: griffin-doc/measure/measure-streaming-sample.md ---
@@ -18,7 +18,7 @@ under the License.
-->
# Measure Stream
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/432#discussion_r224948471
--- Diff: griffin-doc/measure/measure-batch-sample.md ---
@@ -18,7 +18,7 @@ under the License.
-->
# Measure Batch Sam
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/434#discussion_r224956533
--- Diff:
measure/src/main/scala/org/apache/griffin/measure/step/builder/dsl/transform/AccuracyExpr2DQSteps.scala
---
@@ -125,14 +126,20
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/434#discussion_r224956554
--- Diff:
measure/src/main/scala/org/apache/griffin/measure/step/builder/dsl/transform/AccuracyExpr2DQSteps.scala
---
@@ -125,14 +126,20
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/434#discussion_r224956565
--- Diff:
measure/src/main/scala/org/apache/griffin/measure/step/builder/dsl/transform/AccuracyExpr2DQSteps.scala
---
@@ -125,14 +126,20
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/435
let me check
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/435
data partition timezone is come from data connector.
https://github.com/apache/incubator-griffin/blob/master/service/src/main/java/org/apache/griffin/core/measure/entity
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/439
LGTM
---
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/440#discussion_r226866574
--- Diff: griffin-doc/deploy/deploy-guide.md ---
@@ -84,14 +84,14 @@ curl -XPUT http://es:9200/griffin -d '
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/441#discussion_r226866655
--- Diff:
service/src/main/java/org/apache/griffin/core/login/LoginServiceLdapImpl.java
---
@@ -48,68 +53,137 @@ Licensed to the Apache
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/441#discussion_r226866713
--- Diff:
service/src/main/java/org/apache/griffin/core/login/LoginServiceLdapImpl.java
---
@@ -48,68 +53,137 @@ Licensed to the Apache
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/442#discussion_r226866876
--- Diff:
service/src/main/java/org/apache/griffin/core/measure/entity/Rule.java ---
@@ -81,6 +81,9 @@ Licensed to the Apache Software
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/442#discussion_r227272894
--- Diff:
service/src/main/java/org/apache/griffin/core/measure/entity/Rule.java ---
@@ -81,6 +81,9 @@ Licensed to the Apache Software
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/445#discussion_r228021021
--- Diff: griffin-doc/deploy/deploy-guide.md ---
@@ -50,7 +50,7 @@ createdb -O quartz
```
Init quartz tables in PostgreSQL using
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/446
Awesome!
---
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/442#discussion_r228217972
--- Diff:
service/src/main/java/org/apache/griffin/core/measure/entity/Rule.java ---
@@ -81,6 +81,9 @@ Licensed to the Apache Software
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/447#discussion_r228419404
--- Diff:
service/src/main/java/org/apache/griffin/core/metastore/hive/HiveMetaStoreServiceImpl.java
---
@@ -99,8 +100,17 @@ public
Github user guoyuepeng commented on a diff in the pull request:
https://github.com/apache/incubator-griffin/pull/447#discussion_r228420528
--- Diff: ui/angular/src/app/measure/create-measure/ac/ac.component.ts ---
@@ -644,48 +644,37 @@ export class AcComponent implements OnInit
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/447
@chemikadze LGTM.
Just rushed out for my meeting. sorry for confusing.
---
Github user guoyuepeng commented on the issue:
https://github.com/apache/incubator-griffin/pull/450
LGTM
---
1 - 100 of 168 matches
Mail list logo