slfan1989 commented on PR #15: URL: https://github.com/apache/paimon-cpp/pull/15#issuecomment-4581381674
> @slfan1989 Hello, `StringUtils` has been merged in #23 . When you have time, could you please rebase your branch and update this PR accordingly? Thanks again for the fix! @lxy-9602 Thanks for the reminder and for reviewing the fix! I have rebased the branch onto the latest main and updated the PR to use StringUtils::ToUpperCase from #23. I also moved FromShortString into row_kind.cpp to avoid adding the StringUtils dependency to the header. Verified the RowKind parsing behavior locally. -- 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]
