You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/23 11:37:11 UTC

[GitHub] [arrow] lidavidm commented on pull request #13215: MINOR: [C++] Fix incorrect Decimal literal conversion in `ir_consumer.cc`

lidavidm commented on PR #13215:
URL: https://github.com/apache/arrow/pull/13215#issuecomment-1134562532

   Thanks for the contribution!
   
   Just a couple general points, but first: IIRC we are favoring Substrait over the Compute IR now, should we instead drop this module? CC @westonpace @bkietz 
   
   We should add tests & create a JIRA for this as it's not really a minor PR (even if the fix is only a few lines of code)


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org