← 输入框|github.com|刷新
Skip to content

[FlinkRunner] Ingestion job tries to connect to every store available in core #20

Closed

Description

@zhilingc

Expected Behavior

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.

Activity

  1. tims commented on Jan 3, 2019

    @tims
    Contributor

    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?

  2. tims commented on Jan 3, 2019

    @tims
    Contributor

    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?

  3. zhilingc commented on Jan 3, 2019

    @zhilingc
    CollaboratorAuthor

    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?

  4. self-assigned this
    on Jan 9, 2019
  5. added a commit that references this issue on Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions