pitrou commented on code in PR #14004:
URL: https://github.com/apache/arrow/pull/14004#discussion_r958543300
##########
cpp/src/arrow/compute/kernels/scalar_set_lookup.cc:
##########
@@ -67,8 +67,11 @@ struct SetLookupState : public KernelState {
auto visit_valid = [&](T v) {
const auto memo_size =
static_cast<int32_t>(memo_index_to_value_index.size());
int32_t unused_memo_index;
- auto on_found = [&](int32_t memo_index) { DCHECK_LT(memo_index,
memo_size); };
- auto on_not_found = [&](int32_t memo_index) {
+ // (capture `memo_index` by value because of ARROW-17567
Review Comment:
Typo, my bad
```suggestion
// (capture `memo_size` by value because of ARROW-17567)
```
--
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]
