Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG] regexp_test and many other test failures #7041

Closed
jlowe opened this issue Nov 10, 2022 · 3 comments
Closed

[BUG] regexp_test and many other test failures #7041

jlowe opened this issue Nov 10, 2022 · 3 comments
Assignees
@jlowe
Labels
bug Something isn't working cudf_dependency An issue or PR with this label depends on a new feature in cudf

Comments

@jlowe
Copy link
Member

jlowe commented Nov 10, 2022 •
edited
Loading

Many tests in regexp_test failed in the nightly build along with other tests. Appears to be related to JNI/cudf code, since rolling back spark-rapids-jni by one day allows the test to pass. All tests that failed:

  • test_cast_timestamp_to_string
  • test_json_ts_formats_round_trip
  • test_str_to_map_expr_with_all_regex_delimiters
  • test_regexp_replace_word
  • test_predefined_character_classes
  • test_regexp_split_unicode_support
  • test_re_replace
  • test_re_replace_repetition
  • test_re_replace_backrefs
  • test_re_replace_anchors
  • test_re_replace_backrefs_escaped
  • test_re_replace_escaped
  • test_re_replace_null
  • test_regex_replace
  • test_regex_replace_chartacter_set_negated
  • test_word_boundaries
  • test_character_classes
  • test_regexp_hexadecimal_digits
  • test_regexp_whitespace
  • test_regexp_horizontal_vertical_whitespace
  • test_regexp_linebreak
  • test_regexp_replace_digit
  • test_window_ride_along
  • test_cast_struct_to_string
@jlowe jlowe added bug Something isn't working ? - Needs Triage Need team to review and classify labels Nov 10, 2022
@jlowe jlowe self-assigned this Nov 10, 2022
@jlowe jlowe added the cudf_dependency An issue or PR with this label depends on a new feature in cudf label Nov 10, 2022
@jlowe
Copy link
Member Author

jlowe commented Nov 10, 2022

Tracked this down to rapidsai/cudf#11927. The tests pass on the commit without this change, holding the plugin source constant.

@jlowe
Copy link
Member Author

jlowe commented Nov 10, 2022

Verified this is fixed by the change pending in rapidsai/cudf#12119.

@pxLi
Copy link
Collaborator

pxLi commented Nov 11, 2022

Confirmed the cudf fix works after new JNI build, closing this ticket

@pxLi pxLi closed this as completed Nov 11, 2022
@sameerz sameerz removed the ? - Needs Triage Need team to review and classify label Nov 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jlowe jlowe

Labels
bug Something isn't working cudf_dependency An issue or PR with this label depends on a new feature in cudf
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jlowe @sameerz @pxLi