Skip to content

Move spark-related parts to a separate repo#1307

Merged
oavdeev merged 13 commits into
feast-dev:masterfrom
oavdeev:remove-everything
Feb 9, 2021
Merged

Move spark-related parts to a separate repo#1307
oavdeev merged 13 commits into
feast-dev:masterfrom
oavdeev:remove-everything

Conversation

@oavdeev
Copy link
Copy Markdown
Collaborator

@oavdeev oavdeev commented Feb 1, 2021

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

Spark-related functionality moved to a separate package `feast-spark`

@oavdeev
wip refactor
dbba616 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@oavdeev
merge master
dfa7b60 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@oavdeev
merge master
26a340f 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@oavdeev
remove jobservice make target
89227ad
@oavdeev
merge master
b31d8eb 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@oavdeev
move validation to feast spark
aebc12f 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@feast-ci-bot
Copy link
Copy Markdown
Collaborator

feast-ci-bot commented Feb 1, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: oavdeev
To complete the pull request process, please assign khorshuheng after the PR has been reviewed.
You can assign the PR to them by writing /assign @khorshuheng in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@oavdeev
merge master
22aa4b9 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@oavdeev oavdeev changed the title [WIP] Move spark-related parts to a separate repo Move spark-related parts to a separate repo Feb 3, 2021
@oavdeev
remove more scripts and tests that were moved to feast-spark
1e1c04b 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@oavdeev
merge master
4c01fdb
@oavdeev
move integration tests out
ba62651 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@oavdeev
remove spark related unit tests
41c793f 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@oavdeev
Merge branch 'master' into remove-everything
c8da664
@oavdeev
fix linter warnings
fee3a98 
Signed-off-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
@feast-ci-bot
Copy link
Copy Markdown
Collaborator

feast-ci-bot commented Feb 9, 2021

@oavdeev: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
test-end-to-end-gcp fee3a98 link /test test-end-to-end-gcp
test-end-to-end-sparkop fee3a98 link /test test-end-to-end-sparkop
test-end-to-end-aws fee3a98 link /test test-end-to-end-aws
test-end-to-end-azure fee3a98 link /test test-end-to-end-azure
python-sdk-integration-test fee3a98 link /test python-sdk-integration-test
test-end-to-end fee3a98 link /test test-end-to-end

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@oavdeev oavdeev merged commit dbe17b4 into feast-dev:master Feb 9, 2021
@oavdeev oavdeev deleted the remove-everything branch February 9, 2021 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@davidheryanto davidheryanto Awaiting requested review from davidheryanto

@khorshuheng khorshuheng Awaiting requested review from khorshuheng

@pyalex pyalex Awaiting requested review from pyalex

@woop woop Awaiting requested review from woop

@zhilingc zhilingc Awaiting requested review from zhilingc

Assignees

No one assigned

Labels

needs-kind release-note size/XXL

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@oavdeev @feast-ci-bot