-
Notifications
You must be signed in to change notification settings - Fork 80
[feat] delta embedding dump upload to PAI Feature Store #603
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
eric-gecheng
wants to merge
23
commits into
alibaba:master
Choose a base branch
from
eric-gecheng:feat/delta_export_optimize
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 14 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
c49f455
basic impl of delta dump upload to feature store
eric-gecheng b697ee5
Merge branch 'master' into feat/delta_export_optimize
eric-gecheng 01481c1
parallelize FeatureStore delta uploads
eric-gecheng e4cffe7
support minute-based delta embedding dumps
eric-gecheng 953a70c
add FeatureStore delta readback checker
eric-gecheng bb4f1d3
put feature store credentials into envs
eric-gecheng 035fc1a
remove redundant delta dump columns
eric-gecheng 2faabcc
export zeros for missing dynamic embedding keys
eric-gecheng 14ba85d
[bugfix] replay restored FeatureStore deltas by generation
eric-gecheng 8eb37f2
[bugfix] synchronize timed delta dump workers
eric-gecheng 0183ec0
[bugfix] use VPC routing for FeatureStore uploads
eric-gecheng 70ee3b8
[bugfix] sanitize distributed export configs
eric-gecheng a611243
[bugfix] synchronize timed delta dump failures across ranks
eric-gecheng cc03f25
[bugfix] reject uneven batches during timed delta dump
eric-gecheng 0af5f74
[bugfix] reject uneven dataloader exhaustion for all multi-rank delta…
eric-gecheng 00a874b
[bugfix] rendezvous FeatureStore failure bit on every distributed dum…
eric-gecheng 736a471
[perf] pipeline the delta dump state rendezvous off the training hot …
eric-gecheng c22051d
[perf] cap rank-zero delta upload memory with streamed reads and spil…
eric-gecheng 36cad98
[perf] reconcile committed delta steps once instead of on every poll
eric-gecheng a2d8967
[bugfix] restrict feature_store_config.endpoint to trusted HTTPS hosts
eric-gecheng 42ee912
fix comment
eric-gecheng b169a98
[bugfix] let parquet delta dump proceed for shared EC/EBC table names
eric-gecheng 2f7dc48
[ci] scope-mark the dynamicemb zero-fill delta dump tests
eric-gecheng File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| feature_store_py @ https://gecheng-rec.oss-accelerate-overseas.aliyuncs.com/software/feature_store_py-2.2.7-py3-none-any.whl | ||
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| # Copyright (c) 2026, Alibaba Group; | ||
| # Licensed under the Apache License, Version 2.0 (the "License"); | ||
| # you may not use this file except in compliance with the License. | ||
| # You may obtain a copy of the License at | ||
| # http://www.apache.org/licenses/LICENSE-2.0 | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. |
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please source this credential-bearing SDK from an organization-controlled registry/release and pin its SHA-256 hash. A mutable wheel URL in a personal bucket makes dependency compromise equivalent to arbitrary code execution with the training job’s FeatureStore credentials.