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

   ### Rationale for this change
   C++ 20 already includes a span implementation so there is no reason to 
maintain a custom one in utils as described in #48588
   
   ### What changes are included in this PR?
   arrow::utils::span definition and tests are removed and usages are replaced 
with std::span. Some span comparisons in tests are replaced from ASSERT_EQ to 
ASSERT_TRUE(std::ranges::equal(span1, span2)) because ASSERT_EQ inernally 
relies on == operator which is not defined for some types that we used to 
compare.
   
   ### Are these changes tested?
   Yes
   
   ### Are there any user-facing changes?
   No


-- 
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