fix: fix err log

[zhoujh01]

Description

Related Issue

Type of Change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactoring (no functional changes)
• Performance improvement
• Test update

Changes Made

Testing

• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• I have tested this on the following platforms:
  • Linux
  • macOS
  • Windows

Checklist

• My code follows the project's coding style
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• Any dependent changes have been merged and published

Screenshots (if applicable)

Additional Notes

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Possible Regression The PR removed the check that returned a default RawDSL({"op": "and", "conds": []}) when merged was None. This could cause _build_scope_filter to return None instead of a valid filter object, potentially breaking downstream code that expects a filter. merged = self._merge_filters(*filters) return merged

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Restore None check with default filter The change removed the check that returns a default RawDSL({"op": "and", "conds": []}) when merged is None. This could cause unexpected behavior if downstream code expects a filter object rather than None. Restore the check for merged being None to maintain the original behavior. openviking/storage/viking_vector_index_backend.py [497-498] merged = self._merge_filters(*filters) +if merged is None: + return RawDSL({"op": "and", "conds": []}) return merged Suggestion importance[1-10]: 8 __ Why: The suggestion correctly identifies that removing the None check could cause unexpected behavior in downstream code expecting a filter object instead of None, addressing a potential regression.	Medium