korowa commented on code in PR #2591: URL: https://github.com/apache/arrow-datafusion/pull/2591#discussion_r882331249
########## dev/build-arrow-ballista.sh: ########## @@ -24,7 +24,7 @@ rm -rf arrow-ballista 2>/dev/null # clone the repo # TODO make repo/branch configurable -git clone https://github.com/apache/arrow-ballista +git clone --branch join_filter https://github.com/korowa/arrow-ballista Review Comment: I guess it should be merged as it is, then. we update datafusion revision in related ballista PR, and after ballista PR is merged we reset this back to ballista master ########## dev/build-arrow-ballista.sh: ########## @@ -24,7 +24,7 @@ rm -rf arrow-ballista 2>/dev/null # clone the repo # TODO make repo/branch configurable -git clone https://github.com/apache/arrow-ballista +git clone --branch join_filter https://github.com/korowa/arrow-ballista Review Comment: I guess it should be merged as it is, then we update datafusion revision in related ballista PR, and after ballista PR is merged we reset this back to ballista master -- 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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
