Skip to content

Conversation

@dhananjay-mk
Copy link
Contributor

No description provided.

@dhananjay-mk dhananjay-mk requested review from Copilot and vatj March 11, 2025 14:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds documentation detailing pre-insert schema validations for online feature groups with common error types and corrective actions.

  • Adds a new documentation section on pre-insert schema validation for online feature groups.
  • Introduces a table outlining key error types, requirements, and suggestions for correction.

@dhananjay-mk dhananjay-mk requested a review from Copilot March 13, 2025 15:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds documentation for pre-insert schema validation errors for online feature groups, providing guidance on how to handle common validation issues.

  • Introduces a new section on pre-insert schema validation.
  • Provides Python and Pandas code examples for handling primary key null values, missing primary key columns, and string length issues.


#### Pre-insert schema validation for online feature groups

The input dataframe can be validated for schema as per the valid online schema data types before online ingestion. The most important checks are mentioned below along with possible corrective actions. It is enabled by setting the keyword argument `validation_options={'run_validation':True}` in the `insert()` API of feature groups.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The validation is enabled by default so maybe you should say how to disable it instead

@dhananjay-mk dhananjay-mk merged commit 81fa642 into logicalclocks:main Apr 24, 2025
1 check passed
@dhananjay-mk dhananjay-mk changed the title add pre-insert schema validation errors [FSTORE-1708] add pre-insert schema validation errors Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants