Skip to content

Collapse Spatial/TemporalGeneric/TemporalQuantile/OneTemporalOneSpatial into SpatioTemporal-only classification #500

Description

@milindsrivastava1997

Part of #497. Depends on the topk-detection and matcher-level nested-query-removal child issues above.

Remove detection/handling of the Spatial, TemporalGeneric, TemporalQuantile, and OneTemporalOneSpatial query pattern types (see asap-common/dependencies/rs/sql_utilities/src/ast_matching/sqlpattern_matcher.rs and the SQL-side matches in asap-query-engine/src/engines/simple_engine/{sql,mod}.rs) so everything folds into SpatioTemporal, making SQL query classification query-type-agnostic.

Required sub-steps:

  • Update asap-planner-rs SQL planner window-size logic (asap-planner-rs/src/planner/window.rs) to drop the now-removed pattern types.
  • Rewrite sql_utilities ast_matching tests (asap-common/dependencies/rs/sql_utilities/src/ast_matching/sqlparser_test.rs) for SpatioTemporal-only classification.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions