allow empty string to select a NoOp write transform

[tims]

This is for #5 .

Note this also allows core to return no warehouse store id and/or no serving store id in the feature specs. Either one, will be replaced with a NoOp.

[woop]

Is it possible to rather have an option to turn off a store type instead of having empty strings?

[tims]

Yes there is. If there is a storage spec that specifies type = "noop" you can use it's store id.

[woop]

Yes there is. If there is a storage spec that specifies type = "noop" you can use it's store id.

I mean an option within the feature specification like useFeatureWarehouse: false

and then omitting the actual store from the spec.

[pradithya]

I mean an option within the feature specification like useFeatureWarehouse: false

Can we have this as separate issue?

[pradithya]

/retest

[woop]

I mean an option within the feature specification like useFeatureWarehouse: false

Can we have this as separate issue?

Done #38

/retest

You need to rebase first @pradithya . You dont have a Makefile in the root of the repo with a test command in it.

[tims]

rebased against master

/retest

[tims]

/retest

[pradithya]

/lgtm

[tims]

/approve

[feast-ci-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tims

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 [tims]

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