Skip to content

fix: add missing parsers after merging v0.23.1#55

Merged
whhe merged 1 commit intooceanbase:mainfrom
whhe:fix-powerrag-parser
Feb 25, 2026
Merged

fix: add missing parsers after merging v0.23.1#55
whhe merged 1 commit intooceanbase:mainfrom
whhe:fix-powerrag-parser

Conversation

@whhe
Copy link
Member

@whhe whhe commented Feb 25, 2026

Summary

Fixes a parser error introduced when merging upstream branches.

Solution Description

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Feb 25, 2026
@whhe whhe requested a review from Copilot February 25, 2026 08:04
@dosubot dosubot bot added the bug Something isn't working label Feb 25, 2026
@dosubot
Copy link

dosubot bot commented Feb 25, 2026

Related Documentation

Checked 8 published document(s) in 1 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a parser error introduced during a merge with upstream branches (v0.23.1). The changes add missing PowerRAG parser registrations and update the distributed locking import pattern for consistency.

Changes:

  • Added imports for three PowerRAG parsers (title, regex, smart) from powerrag.app module
  • Registered the three PowerRAG parsers in the FACTORY dictionary mapping ParserType enum values to parser implementations
  • Updated the import and usage of distributed_lock function instead of RedisDistributedLock class for consistency with the codebase pattern

@whhe whhe requested a review from Zhangg7723 February 25, 2026 09:18
@whhe whhe merged commit 61e3a2e into oceanbase:main Feb 25, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants