Skip to content

[VL] Spark 4.1: Velox split function returns incorrect results with limit parameter (SPARK-49968) #11913

Open

Description

@baibaichen

Backend

VL (Velox)

Bug description

split('hello', '', 1) returns ["h"] in Velox but should return ["hello"]. This is a Velox C++ re2-based split implementation issue when the limit parameter is specified. Spark 4.1 added the limit parameter test via SPARK-49968.

Spark 4.1 only.

Parent issue: #11550

Impact

Suite / Item Status spark40 spark41
GlutenCollationRegexpExpressionsSuite Entire suite TODO (1 failure) 🟢 🔴
GlutenRegexpExpressionsSuite .exclude("SPLIT") 🟢 🔴
VeloxStringFunctionsSuite testWithSpecifiedSparkVersion skips 4.x 🟢 🔴

References

Activity

  1. Yao-MR commented on Apr 20, 2026

    @Yao-MR
    Contributor

    i will take this issue

  2. brijrajk commented on May 29, 2026

    @brijrajk
    Contributor

    Hi @Yao-MR, are you still planning to work on this? Happy to pick it up if you've moved on.

  3. Yao-MR commented on Jun 1, 2026

    @Yao-MR
    Contributor

    Hi @Yao-MR, are you still planning to work on this? Happy to pick it up if you've moved on.

    yes, the related change on velox side is in review, will move on here once the change on the velox is merged

  4. rdtr commented on Oct 3, 2026

    @rdtr
    Contributor

    The Velox fix (facebookincubator/velox#17327) was closed, so I've taken it over in facebookincubator/velox#19335, crediting @Yao-MR as a co-author. Once it lands, I'll pass spark.sql.legacy.truncateForEmptyRegexSplit to Velox in Gluten.

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

    bugSomething isn't working

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions