lriggs opened a new pull request, #36184:
URL: https://github.com/apache/arrow/pull/36184

   ### Rationale for this change
   
   substring_index("Abc.DE.fGh", '.', -2) returns "fGh" but it should return 
"DE.fGh" (ie starting from the second occurrence of the delimiter from the 
right). The proposed behavior matches the behavior of other databases.
   
   ### What changes are included in this PR?
   
   Fixed reverse index calculation and updated unit tests.
   
   ### Are these changes tested?
   
   Yes, unit tests and integration testing.
   
   ### Are there any user-facing changes?
   
   Function behavior change with substring_index function.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to