Skip to content

[Query] Clean up and decouple internal Kgram engine modules in corese-next #617

Closed

Description

@remiceres

Summary

Clean up, modernize, and clarify internal module boundaries inside core.next.query.impl.engine, removing dead code, harmonizing exception hierarchies, and enforcing strict typing.

Objectives and Technical Scope

1. Package Reorganization and Boundary Clarification

• Review internal packages in core.next.query.impl.engine (eval, event, filter, model, path, pattern, solution, sorter, spi, storage).
• Clarify boundaries between planning, optimization, and runtime execution.

2. Dead Code and Legacy Pruning

• Identify and remove unused classes, deprecated methods, and redundant abstractions left over from the initial engine import.
• Modernize variable names and internal terminology to match contemporary standards.

3. Unified Error Handling and Typing

• Standardize query evaluation exceptions into a coherent hierarchy rooted in core.next.query.api.exception.
• Ensure strict, pervasive usage of DatatypeValue across all engine internal state.

Acceptance Criteria

[ ] Dead code and unused classes in next.query.impl.engine are eliminated.
[ ] Exception hierarchy is clean, coherent, and documented.
[ ] Architecture boundary tests (NextModuleBoundaryTest) pass with zero violations.
[ ] 100% test pass rate across existing unit and integration tests.

Activity

  1. added
    refactoringInternal code restructuring and architecture cleanup
    on Sep 29, 2026
  2. self-assigned this
    on Sep 29, 2026
  3. remiceres commented on Oct 19, 2026

    @remiceres
    ContributorAuthor

    Closed via PR #632. All acceptance criteria met:

    • Dead code, unused classes, and legacy KGRAM baggage in next.query.impl.engine eliminated.
    • Exception hierarchy standardized and documented.
    • Architecture boundary tests (NextModuleBoundaryTest) pass with zero violations.
    • 100% test pass rate across all 3,372 Corese tests and isolated checkNext suite.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

refactoringInternal code restructuring and architecture cleanup

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions