Infer features for on demand feature views, support multiple output features

[achals]

Signed-off-by: Achal Shah achals@gmail.com

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

Infer features for on demand feature views, support multiple output features 

[codecov-commenter]

Codecov Report

Merging #1845 (bb914b6) into master (b509092) will increase coverage by 0.32%.
The diff coverage is 95.83%.

@@            Coverage Diff             @@
##           master    #1845      +/-   ##
==========================================
+ Coverage   84.52%   84.84%   +0.32%     
==========================================
  Files          90       91       +1     
  Lines        6837     6924      +87     
==========================================
+ Hits         5779     5875      +96     
+ Misses       1058     1049       -9     

Flag	Coverage Δ
integrationtests	84.77% <95.83%> (+0.32%)	⬆️
unittests	63.41% <40.83%> (-0.40%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...fline_store/test_universal_historical_retrieval.py	100.00% <ø> (ø)
sdk/python/feast/type_map.py	81.25% <62.50%> (-1.61%)	⬇️
sdk/python/feast/on_demand_feature_view.py	91.78% <96.00%> (+2.19%)	⬆️
sdk/python/feast/feature_store.py	93.76% <97.14%> (+0.62%)	⬆️
sdk/python/feast/errors.py	69.15% <100.00%> (+1.19%)	⬆️
sdk/python/feast/feature.py	75.00% <100.00%> (ø)
sdk/python/feast/infra/offline_stores/file.py	97.43% <100.00%> (ø)
...python/feast/infra/offline_stores/offline_utils.py	91.20% <100.00%> (+0.09%)	⬆️
sdk/python/feast/infra/provider.py	89.25% <100.00%> (+0.27%)	⬆️
...tegration/feature_repos/universal/feature_views.py	96.00% <100.00%> (+0.16%)	⬆️
... and 6 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b509092...bb914b6. Read the comment docs.

[adchia]

Nice! Looks like there is some weird rebasing that happened re the README

[adchia]

is this a pandas type? couldn't find documentation for Timestamp

[achals]

oops - I think this is a protobuf type. Fixing.

[adchia]

nit: view_name, feature_name = feature_ref.split(":") I tihnk should work

[achals]

cool

[achals]

Nice! Looks like there is some weird rebasing that happened re the README

This is a precommit hook that ran on the README. my guess is that it was added after the README changes were committed and it wasn't run on all existing files.

[adchia]

/lgtm

[feast-ci-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: achals, adchia

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [achals,adchia]

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