Skip to content

Native binary overloads: support STARTSWITH, ENDSWITH and ELT #218

Description

@liuyongvs

Rechecked on 2026-09-20 against canonical main 89da01cdad7342c4d825e36eaf4e8dc73bc7af9f, released Apache Flink 2.2.1, JDK 17, default compatibility settings. Each expression was run against independent stock-Flink and StreamFusion-enabled streaming environments with bounded runtime DataStream input (parallelism 1), so source values cannot be folded away. Results and resolved result types agree through Flink fallback; this is a native support gap, not a demonstrated wrong-result bug.

The character overloads do not cover BYTES arguments. Adapted StringFunctionsITCase SQL and minimal runtime probes confirm these remaining binary forms.

Runtime columns v BYTES=X'01020304', p BYTES=X'0102', q BYTES=X'0304':

SELECT STARTSWITH(v,p) FROM src; -- TRUE
SELECT ENDSWITH(v,q) FROM src;   -- TRUE
SELECT ELT(2,p,q) FROM src;      -- X'0304'

Recorded reasons:

  • Calc: STARTSWITH requires 2..2 character arguments
  • Calc: ENDSWITH requires 2..2 character arguments
  • Calc: ELT requires an INTEGER index and character values

The BOOLEAN/BOOLEAN/BYTES outputs match released Flink through fallback with zero native substitutions. Cover arbitrary byte values including zero/high-bit bytes, empty values, NULLs, index bounds and sliced buffers; do not decode binary operands as text.

Suggested priority: P3, after STRING, exact numeric and temporal scalar coverage.

Acceptance: add runtime-source SQL parity tests that assert native Calc execution as well as values and resolved types; include NULLs and boundary/error cases; retain safe fallback for unverified forms and update docs/operators/calc-filter.md with the admitted scope.

Latest-main validation: rebuilt the debug native library from this commit and ran 562 targeted runtime SQL cases covering these support gaps and adjacent controls. This issue group contains 19 successful comparisons that still record explicit fallback. Host-rejected/both-error cases are excluded from that count. This validates released Flink 2.2.1; it does not claim execution of the Flink 1.18 profile. No implementation is included.

Related: PR #44 documented narrower admitted overloads and the remaining fallback; this ticket tracks the unimplemented forms.

No activity

Activity on this issue will appear here.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions