Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Griffin 1.0.0 #646

Closed
wants to merge 15 commits into from
Closed

Griffin 1.0.0 #646

wants to merge 15 commits into from

Conversation

guoyuepeng
Copy link
Contributor

No description provided.

guoyuepeng and others added 8 commits September 30, 2022 15:01
* init griffin new workflow

* model data quality request

Co-authored-by: guoyp <guoyp@guoypdeMacBook-Pro.local>
* Update GriffinWebApplication.java

* test  update

* worker

* first commit

* dev

* recording流程处理

* Griffin 1.0.0 dev (apache#611)

* init griffin new workflow

* model data quality request

Co-authored-by: guoyp <guoyp@guoypdeMacBook-Pro.local>

Co-authored-by: Warden <wangding2587@gmail.com>
Co-authored-by: William Guo <guoyp@apache.org>
Co-authored-by: guoyp <guoyp@guoypdeMacBook-Pro.local>
Copy link

github-actions bot commented Apr 7, 2024

Automated Message: This PR is being labelled as stale and will be closed in next 15 days due to lack of activity. To avoid this push new commits or ask the committers for a review/ resolution.

pom.xml Show resolved Hide resolved
@guoyuepeng
Copy link
Contributor Author

guoyuepeng commented Apr 11, 2024

we need to refactor our scheduler so that we might leverage customer's existing scheduler engines.
most of time, most customers maintain one scheduler engine.

@guoyuepeng
Copy link
Contributor Author

existing trigger are usually like

public interface Trigger {
@nullable
Date nextExecutionTime(TriggerContext triggerContext);
}

but we need something like

Trigger with CitiesDimension with SeasonDimension

those cities are dynamically injected

@toyboxman
Copy link
Member

LGTM.

Let's merge ASAP, then all paticipants of the commnity can continue to contribute and bring out the new grffin.

Thanks

@guoyuepeng
Copy link
Contributor Author

LGTM.

Let's merge ASAP, then all paticipants of the commnity can continue to contribute and bring out the new grffin.

Thanks

a long way to go, let's make it faster.

@toyboxman
Copy link
Member

Hello, @guoyuepeng

Could you merge this, let's start with latest basis.

Copy link

Automated Message: This PR is being labelled as stale and will be closed in next 15 days due to lack of activity. To avoid this push new commits or ask the committers for a review/ resolution.

Copy link

github-actions bot commented Jun 8, 2024

Automated Message: We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the 'no-pr-activity' tag!

@github-actions github-actions bot closed this Jun 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants