allow https url for spark ingestion jar

[jklegar]

Signed-off-by: Jacob Klegar [email protected]

What this PR does / why we need it: previously if the spark ingestion jar was a https url (which is the default), ingestion would fail because Feast would try to read and upload the jar as if the url were a local path

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

NONE

[woop]

@jklegar so this change doesn't apply to dataproc?

[jklegar]

I believe on dataproc https should already work due to

feast/sdk/python/feast/pyspark/launchers/gcloud/dataproc.py

Line 278 in eaaf233

if not os.path.isfile(file_path):

[oavdeev]

/retest

[oavdeev]

/test test-end-to-end-aws

[jklegar]

/test test-end-to-end-aws

[feast-ci-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

Needs approval from an approver in each of these files:

• OWNERS

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