Skip to content

Feat: add memory function by go#13754

Merged
JinHai-CN merged 12 commits intoinfiniflow:mainfrom
dcc123456:feature/go-0320
Mar 27, 2026
Merged

Feat: add memory function by go#13754
JinHai-CN merged 12 commits intoinfiniflow:mainfrom
dcc123456:feature/go-0320

Conversation

@dcc123456
Copy link
Copy Markdown
Contributor

What problem does this PR solve?

Feat: Add Memory function by go

Type of change

  • New Feature (non-breaking change which adds functionality)

…arks them as TODO.

`refactor(service):` Deletes code related to `CanvasService` and `TaskService`.

`refactor(handler):` Marks unimplemented memory message handling functions as TODO and returns an error.

`refactor(service):` Changes the type of `TenantEmbdID` and `TenantLLMID` to string.
…nterfaces.

Removes the unimplemented `UpdateDocument`, `DeleteByQuery`, and `UpdateByQuery` methods from the `DocEngine` interface.
Also removes the corresponding empty implementations from the Infinity and Elasticsearch engines.
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Mar 24, 2026
@dcc123456 dcc123456 added the ci Continue Integration label Mar 24, 2026
@dcc123456 dcc123456 marked this pull request as draft March 24, 2026 03:00
@dcc123456 dcc123456 marked this pull request as ready for review March 24, 2026 03:00
@yingfeng yingfeng changed the title Feat: Add Memory function by go Feat: Add Memory function by go Mar 24, 2026
@yingfeng yingfeng changed the title Feat: Add Memory function by go Feat: add Memory function by go Mar 24, 2026
@yingfeng yingfeng changed the title Feat: add Memory function by go Feat: add memory function by go Mar 24, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.72%. Comparing base (e705ac6) to head (a2a93d1).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #13754   +/-   ##
=======================================
  Coverage   96.72%   96.72%           
=======================================
  Files          10       10           
  Lines         702      702           
  Branches      112      112           
=======================================
  Hits          679      679           
  Misses          5        5           
  Partials       18       18           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JinHai-CN JinHai-CN merged commit 8a9bbf3 into infiniflow:main Mar 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continue Integration size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants