Skip to content

[EN] Add warnings for key types and feature types (since they are different) #766

@xiaoyongzhu

Description

@xiaoyongzhu

Willingness to contribute

No. I cannot contribute a bug fix at this time.

Feathr version

0.8

System information

  • OS Platform and Distribution (e.g., Linux Ubuntu 20.0):
  • Python version:
  • Spark version, if reporting runtime issue:

Describe the problem

Currently the Feathr key type and the feature type is different and users usually mix those (we've seen it multiple times). So we should:

  1. Add type checks for the type of key (ValueType.INT32) and feature types (from feathr import INT32)
  2. give warnings out if users mix those

Tracking information

No response

Code to reproduce bug

No response

What component(s) does this bug affect?

  • Python Client: This is the client users use to interact with most of our API. Mostly written in Python.
  • Computation Engine: The computation engine that execute the actual feature join and generation work. Mostly in Scala and Spark.
  • Feature Registry API: The frontend API layer supports SQL, Purview(Atlas) as storage. The API layer is in Python(FAST API)
  • Feature Registry Web UI: The Web UI for feature registry. Written in React

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementFurther enhancement to improve usability

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions