Hi Rossi, I think for small PRs like this it is fine to just prefix your PR with “MINOR” and not have an associated issue.
Sasha > On Apr 13, 2023, at 10:48 AM, Ruoxi Sun <zanmato1...@gmail.com> wrote: > > Hi Sasha, thanks for confirming. Wondering if I should file a github issue > for this kind of trivial fix? > > Rossi > > > Sasha Krassovsky <krassovskysa...@gmail.com > <mailto:krassovskysa...@gmail.com>> 于2023年4月14日周五 01:44写道: >> Hi Rossi, >> That’s a good catch! I _think_ the compiler will automatically emit the move >> because it sees we’re copying from an object that’ll never be used again >> [1], but adding the std::move would be good just to remove any ambiguity. Go >> ahead and make the PR! >> >> Sasha >> >> Move, simply >> herbsutter.com >> >> <https://herbsutter.com/2020/02/17/move-simply/>Move, simply >> <https://herbsutter.com/2020/02/17/move-simply/> >> herbsutter.com <https://herbsutter.com/2020/02/17/move-simply/> >> <https://herbsutter.com/2020/02/17/move-simply/> >> >>> 13 апр. 2023 г., в 10:17, Ruoxi Sun <zanmato1...@gmail.com >>> <mailto:zanmato1...@gmail.com>> написал(а): >>> >>> Hi folks, when reading the swiss join code, I just noticed a small piece >>> probably missing a `std::move()` call. >>> >>> See here: >>> https://github.com/zanmato1984/arrow/commit/10f43c357db7a0287c642a23e78027cb9cde6f25 >>> >>> If so, I think I can proceed to PR it. >>> >>> Thanks. >>> >>> *Rossi Sun*