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/03/08 17:08:59 UTC

[GitHub] [arrow-datafusion] alamb opened a new pull request #1958: fix logical conflict with protobuf

alamb opened a new pull request #1958:
URL: https://github.com/apache/arrow-datafusion/pull/1958


   https://github.com/apache/arrow-datafusion/pull/1941 introduced a small logical conflict with  https://github.com/apache/arrow-datafusion/pull/1887 which results in a compile error on master:
   
   https://github.com/apache/arrow-datafusion/runs/5467727177?check_suite_focus=true
   
   ```
   error: failed to run custom build command for `ballista-core v0.6.0 (/__w/arrow-datafusion/arrow-datafusion/ballista/rust/core)`
   
   Caused by:
     process didn't exit successfully: `/github/home/target/debug/debug/build/ballista-core-8af276defa9847ff/build-script-build` (exit status: 1)
     --- stdout
     cargo:rerun-if-env-changed=FORCE_REBUILD
     cargo:rerun-if-changed=proto/ballista.proto
     cargo:rerun-if-changed=proto/datafusion.proto
   
     --- stderr
     Error: "protobuf compilation failed: protoc failed: ballista.proto:402:3: \"LogicalExprNode\" is not defined.\n"
   warning: build failed, waiting for other jobs to finish...
   error: build failed
   Error: Process completed with exit code 101.
   ```
   
   This PR fixes the problem
   


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

[GitHub] [arrow-datafusion] alamb commented on pull request #1958: fix logical conflict with protobuf

Posted by GitBox <gi...@apache.org>.
alamb commented on pull request #1958:
URL: https://github.com/apache/arrow-datafusion/pull/1958#issuecomment-1062060520


   Since this seems likely to be uncontroversial (and fixes something I broke 😢 ) merging it in


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

[GitHub] [arrow-datafusion] alamb merged pull request #1958: fix logical conflict with protobuf

Posted by GitBox <gi...@apache.org>.
alamb merged pull request #1958:
URL: https://github.com/apache/arrow-datafusion/pull/1958


   


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