You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[FlinkRunner] Ingestion job tries to connect to every store available in core #20
Should only connect to the relevant stores specified in the FeatureSpec.
Current Behavior
Will pull and attempt to connect to all the stores once the job is sent to the remote Flink cluster. If there are stores it cannot connect to, the entire job will fail.
Steps to reproduce
Register a storage that cannot be reached (e.g. 1.2.3.4:6379)
Run a job (not using that store) on a remote flink cluster
Possible Solution
Since we only pull specs at the graph building stage now, ingestion shouldn't have to retrieve and cache all available stores, only the stores it needs.
I thought you said that this is unrelated to ingestion in a previous discussion... but ingestion calls on startup:
specService.getAllStorageSpecs();
Even though it might not need them all. We could change this so that it only requests the serving and warehouse stores listed in the relevant feature specs if you like. Is that the plan for this issue?
I thought you said that this is unrelated to ingestion in a previous discussion... but ingestion calls on startup:
specService.getAllStorageSpecs();
Even though it might not need them all. We could change this so that it only requests the serving and warehouse stores listed in the relevant feature specs if you like. Is that the plan for this issue?
Yes, I think that is ideal.
Question though.. what is trying to connect to the stores? I didn't think that ingestion does if no features are using it. So is core doing that?
No; it is ingestion that fails if there is a store present in the registry that it cannot connect to. Are you able to replicate the error?
Activity
tims commentedon Jan 3, 2019
I thought you said that this is unrelated to ingestion in a previous discussion... but ingestion calls on startup:
Even though it might not need them all. We could change this so that it only requests the serving and warehouse stores listed in the relevant feature specs if you like. Is that the plan for this issue?
tims commentedon Jan 3, 2019
Question though.. what is trying to connect to the stores? I didn't think that ingestion does if no features are using it. So is core doing that?
zhilingc commentedon Jan 3, 2019
Yes, I think that is ideal.
No; it is ingestion that fails if there is a store present in the registry that it cannot connect to. Are you able to replicate the error?
Merge pull request feast-dev#20 from dmartinol/feast-rbac