feat: Support for StringSplit

[Shekharrajak]

Fixes #2707

We have scalar function:
https://datafusion.apache.org/user-guide/sql/scalar_functions.html#string-to-array

[comphead]

Thanks @Shekharrajak for you contribution, please add a function to the fuzztesting kit, similar to #2755

[mbutrovich]

In the past I think we've encountered differences in Java and Rust's regex engines wrt graphemes. Could we get some larger UTF-8 characters in the tests?

[andygrove]

In the past I think we've encountered differences in Java and Rust's regex engines wrt graphemes. Could we get some larger UTF-8 characters in the tests?

We probably need to fall back to Spark unless this config is enabled:

  val COMET_REGEXP_ALLOW_INCOMPATIBLE: ConfigEntry[Boolean] =
    conf("spark.comet.regexp.allowIncompatible")
      .category(CATEGORY_EXEC)
      .doc("Comet is not currently fully compatible with Spark for all regular expressions. " +
        s"Set this config to true to allow them anyway. $COMPAT_GUIDE.")
      .booleanConf
      .createWithDefault(false)

[Shekharrajak]

Thanks @Shekharrajak for you contribution, please add a function to the fuzztesting kit, similar to #2755

Thanks! Added in commit 8eddd29

[Shekharrajak]

In the past I think we've encountered differences in Java and Rust's regex engines wrt graphemes. Could we get some larger UTF-8 characters in the tests?

Added tests 987b646

[Shekharrajak]

In the past I think we've encountered differences in Java and Rust's regex engines wrt graphemes. Could we get some larger UTF-8 characters in the tests?

We probably need to fall back to Spark unless this config is enabled:

  val COMET_REGEXP_ALLOW_INCOMPATIBLE: ConfigEntry[Boolean] =
    conf("spark.comet.regexp.allowIncompatible")
      .category(CATEGORY_EXEC)
      .doc("Comet is not currently fully compatible with Spark for all regular expressions. " +
        s"Set this config to true to allow them anyway. $COMPAT_GUIDE.")
      .booleanConf
      .createWithDefault(false)

How can we check if it is not falling back to Spark's JVM execution? @andygrove

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.46%. Comparing base (f09f8af) to head (bcb6ed4).
⚠️ Report is 708 commits behind head on main.

Additional details and impacted files

@@              Coverage Diff              @@
##               main    #2772       +/-   ##
=============================================
- Coverage     56.12%   45.46%   -10.66%     
- Complexity      976     1206      +230     
=============================================
  Files           119      157       +38     
  Lines         11743    14124     +2381     
  Branches       2251     2365      +114     
=============================================
- Hits           6591     6422      -169     
- Misses         4012     6683     +2671     
+ Partials       1140     1019      -121     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.