Skip to content

[FSTORE-2034] Fix managed FG#964

Open
bubriks wants to merge 9 commits into
logicalclocks:mainfrom
bubriks:FSTORE-2034
Open

[FSTORE-2034] Fix managed FG#964
bubriks wants to merge 9 commits into
logicalclocks:mainfrom
bubriks:FSTORE-2034

Conversation

@bubriks
Copy link
Copy Markdown
Contributor

@bubriks bubriks commented May 21, 2026

This PR adds/fixes/changes...

  • please summarize your changes to the code
  • and make sure to include all changes to user-facing APIs

JIRA Issue: -

Priority for Review: -

Related PRs: -

How Has This Been Tested?

  • Unit Tests
  • Integration Tests
  • Manual Tests on VM

Checklist For The Assigned Reviewer:

- [ ] Checked if merge conflicts with master exist
- [ ] Checked if stylechecks for Java and Python pass
- [ ] Checked if all docstrings were added and/or updated appropriately
- [ ] Ran spellcheck on docstring
- [ ] Checked if guides & concepts need to be updated
- [ ] Checked if naming conventions for parameters and variables were followed
- [ ] Checked if private methods are properly declared and used
- [ ] Checked if hard-to-understand areas of code are commented
- [ ] Checked if tests are effective
- [ ] Built and deployed changes on dev VM and tested manually
- [x] (Checked if all type annotations were added and/or updated appropriately)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  python/hsfs
  storage_connector.py 428-434
  python/hsfs/core
  data_source.py 216, 222, 226
  external_feature_group_engine.py
Project Total  

This report was generated by python-coverage-comment-action

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.

1 participant