Merged
Conversation
Aditi2424
temporarily deployed
to
auto-approve
GitHub Actions
Inactive
mollyheamazon
added a commit
that referenced
this pull request
Feb 27, 2026
* Feature store v3 (#5490) * feat: Add Feature Store Support to V3 * Add feature store tests --------- Co-authored-by: adishaa <adishaa@amazon.com> * Changes: - Add feature_store_functions_report.md documenting all 63 functions across feature_store module (excluding feature_processor) - Add comprehensive unit tests for get_feature_group_as_dataframe: * Session handling (provided, from region, from role) * Error cases (missing session/region, missing event_time) * Latest ingestion logic with event time * Query string manipulation and table placeholder * Verbose and silent logging modes * Kwargs passing to as_dataframe - Add comprehensive unit tests for prepare_fg_from_dataframe_or_file: * DataFrame and file path input handling * Session/region/role configuration * Record ID creation and validation * Event ID creation with timestamp * Duplicate record detection * Column name formatting * CSV kwargs passing * Feature definition loading * Fix * Fix * fix * Integ tests * Fix * Add feature store telemetry * Bug fix for GetRecordResponse not printable and error message for ingest_dataframe * Fix: DatasetBuilder.to_dataframe() docstring and instantiation * Fix unit test failures * fix unit test --------- Co-authored-by: Aditi Sharma <165942273+Aditi2424@users.noreply.github.com> Co-authored-by: adishaa <adishaa@amazon.com> Co-authored-by: Molly He <mollyhe@amazon.com>
BassemHalim
pushed a commit
to BassemHalim/sagemaker-python-sdk
that referenced
this pull request
Feb 27, 2026
* feat: Add Feature Store Support to V3 * Add feature store tests --------- Co-authored-by: adishaa <adishaa@amazon.com>
BassemHalim
pushed a commit
to BassemHalim/sagemaker-python-sdk
that referenced
this pull request
Feb 28, 2026
* feat: Add Feature Store Support to V3 * Add feature store tests --------- Co-authored-by: adishaa <adishaa@amazon.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.