Tags: infiniflow/ragflow
Tags
chore(rag/app): remove stray debug print() calls (#17943) chore(rag/app): remove stray debug print() calls Two hot-path debug print() calls were leaking content/error text to stdout in production code paths. * rag/app/naive.py: TxtParser branch in chunk() was printing the entire parsed sections list (formatted via repr()) wrapped in 150-char banner lines. For large text documents (e.g. a 1000+-page book ingest) this dumped tens of thousands of lines per ingest into the docker logs. Replaced with a structured `logging.info("TxtParser produced %d sections for %s", len(sections), filename)` so the parse count is still observable without the content leak. * rag/app/presentation.py: Pdf.position parsing had a debug `print(f"Error parsing position: {e}")` inside an except clause in the ingest hot path. Replaced with `logging.warning(f"Error parsing position in {filename}: {e}")` to match the file's existing logging pattern and add filename context. Both call sites already had logging imported; no new imports added. logging was used throughout the surrounding code in the same logging.{info,warning,error}(...) style.
fix: embedded/shared agent chat fails with retrieval query unmarshal … …error (#17831)
Go: add soft fingerprint framework (#17837) Signed-off-by: Jin Hai <haijin.chn@gmail.com>
refactor(ingestion/task): extract index-doc mapping into task/indexdo… …c package (#17749) ## Summary Extract the pipeline-output → search-engine index document mapping helpers out of the `task` package into a dedicated, dependency-light leaf package `internal/ingestion/task/indexdoc`. These functions are pure transforms (they only depend on `common`/`utility`) and are not task-orchestration concerns: - `NormalizeChunks`, `DeepCopyChunks` (was unexported `deepCopyChunks`), `toChunkMaps` → `indexdoc/normalize.go` - `ProcessChunksForPipeline`, `RenameTextToContentWithWeight`, `GetEmbeddingTokenConsumption`, `cleanupConsumedChunkFields`, `mergeChunkMetadata`, `processChunkPositions`, `AggregateTableDocMetadata`, `resolveTableColumnConfig` → `indexdoc/process.go` - `AddPositions` → `indexdoc/position.go` - `EmbeddingTokenConsumptionKey` constant → `indexdoc/constants.go` (task/constants.go keeps only `GRAPH_RAPTOR_FAKE_DOC_ID`) Call sites in `pipeline_executor.go` and `golden_compare.go` now reference the `indexdoc` package; package-task tests qualify the moved symbols. ## Why The `task` package had grown into a "orchestration + pure mapping + debug" mix. Splitting the pure mapping helpers into a leaf package sharpens package boundaries, removes a misleading top-level `ingestion/chunk` candidate (there are already `parser/chunk` and `service/chunk`), and lets the golden tool / future reuse pull in the mapping logic without dragging in `task`'s `dao`/`engine`/`service` dependency graph (Go subpackage import does not pull in the parent). ## Test plan - `build.sh --test ./internal/ingestion/task/...` — **green** (task 4.7s, indexdoc 0.007s), matching the pre-change baseline. - `gofmt` clean; `build.sh` builds both `ragflow-cli` and `ragflow_server` successfully. - Integration/E2E tiers are delegated to CI (need real MySQL/MinIO/ES services). Note: `pipeline_e2e_test.go` has a **pre-existing** compile error (`server.ElasticsearchConfig` / `server.InfinityConfig` are now defined under `internal/server/config/`, not re-exported by `internal/server`). This is unrelated to this change — the diff to that file is only the added `indexdoc` import and the qualified `EmbeddingTokenConsumptionKey` reference.
GO CLI: modify enterprise dao functions (#17812) GO CLI: modify enterprise dao functions
GO CLI: add empty enterprise dao functions (#17795) GO CLI: add empty enterprise dao new functions
PreviousNext