Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
682e314
feat: Add dbt integration for importing models as FeatureViews (#3335)
YassinNouh21 Jan 10, 2026
d20962d
fix: Address mypy and ruff lint errors in dbt integration
YassinNouh21 Jan 10, 2026
354f921
fix: Address ruff lint errors in dbt unit tests
YassinNouh21 Jan 10, 2026
5ca317c
style: Format dbt files with ruff
YassinNouh21 Jan 10, 2026
a301f83
fix: Remove unused dbt-artifacts-parser import and fix enum import
YassinNouh21 Jan 10, 2026
460810c
feat: Use dbt-artifacts-parser for typed manifest parsing
YassinNouh21 Jan 10, 2026
b398368
fix: Add graceful fallback for dbt-artifacts-parser validation errors
YassinNouh21 Jan 10, 2026
2f777ff
fix: Skip dbt tests when dbt-artifacts-parser is not installed
YassinNouh21 Jan 10, 2026
86fb951
refactor: Simplify parser to rely solely on dbt-artifacts-parser
YassinNouh21 Jan 10, 2026
a17b50c
ci: Add dbt-artifacts-parser to unit test dependencies
YassinNouh21 Jan 10, 2026
55174a5
fix: Address Copilot code review comments for dbt integration
YassinNouh21 Jan 10, 2026
e4ba00a
fix: Only add ellipsis to truncated descriptions
YassinNouh21 Jan 10, 2026
01730a8
style: Format dbt files with ruff
YassinNouh21 Jan 10, 2026
8a06b83
fix: Convert doctest examples to code blocks to avoid CI failures
YassinNouh21 Jan 10, 2026
fb40e93
fix: Add dbt-artifacts-parser to feast[ci] and update requirements
YassinNouh21 Jan 11, 2026
53932ff
docs: Add dbt integration documentation
YassinNouh21 Jan 11, 2026
972fc96
docs: Add alpha warning to dbt integration documentation
YassinNouh21 Jan 14, 2026
b2901f4
fix: Add dbt-artifacts-parser to CI_REQUIRED dependencies
YassinNouh21 Jan 14, 2026
fe253c1
fix: Add defensive Array.base_type handling with logging
YassinNouh21 Jan 14, 2026
ed2c291
docs: Add comment explaining ImageBytes/PdfBytes exclusion
YassinNouh21 Jan 14, 2026
7a50c73
fix: Move imports to top of file to resolve linter errors
YassinNouh21 Jan 16, 2026
c4ad283
Merge branch 'master' into feat/dbt-feast-integration-3335-clean
franciscojavierarceo Jan 16, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
docs: Add comment explaining ImageBytes/PdfBytes exclusion
Add clarifying comment in type_map explaining why ImageBytes and
PdfBytes are not included in the dbt type mapping. While these types
exist in Feast, dbt manifests only expose generic BYTES type without
semantic information to distinguish between regular bytes, images, or
PDFs.

Example: A dbt model with image and PDF columns both appear as
'BYTES' in the manifest, making ImageBytes/PdfBytes types unmappable
from dbt artifacts.

Addresses feedback from PR #5827 review (franciscojavierarceo).

Signed-off-by: yassinnouh21 <yassinnouh21@gmail.com>
  • Loading branch information
YassinNouh21 committed Jan 14, 2026
commit ed2c2919850a3350f85d75c8f3ee10899f775f21
4 changes: 3 additions & 1 deletion sdk/python/feast/dbt/codegen.py
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,9 @@ def _get_feast_type_name(feast_type: Any) -> str:
base_type_name = _get_feast_type_name(base_type)
return f"Array({base_type_name})"

# Map type objects to their names
# Map type objects to their names.
# Note: ImageBytes and PdfBytes are excluded since dbt manifests only expose
# generic BYTES type without semantic information about binary content.
type_map = {
String: "String",
Int32: "Int32",
Expand Down
Loading