You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2021/05/11 19:34:33 UTC

[GitHub] [calcite-avatica] julianhyde opened a new pull request #144: [CALCITE-4575] Remove Gradle wrapper from source distribution

julianhyde opened a new pull request #144:
URL: https://github.com/apache/calcite-avatica/pull/144


   Please review, but do not merge.
   
   @F21, I think you should merge this (with corrections, if necessary) during your process, as release manager, to create 1.18.0 RC1.


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

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

[GitHub] [calcite-avatica] julianhyde commented on pull request #144: [CALCITE-4575] Remove Gradle wrapper from source distribution

Posted by GitBox <gi...@apache.org>.
julianhyde commented on pull request #144:
URL: https://github.com/apache/calcite-avatica/pull/144#issuecomment-839316811


   I have added a commit with a fix for https://issues.apache.org/jira/browse/CALCITE-4576. Please review that also.


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

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

[GitHub] [calcite-avatica] F21 commented on pull request #144: [CALCITE-4575] Remove Gradle wrapper from source distribution

Posted by GitBox <gi...@apache.org>.
F21 commented on pull request #144:
URL: https://github.com/apache/calcite-avatica/pull/144#issuecomment-839236822


   Thanks @julianhyde I'll test, confirm and merge this during the release.


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

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

[GitHub] [calcite-avatica] F21 merged pull request #144: [CALCITE-4575] Remove Gradle wrapper from source distribution

Posted by GitBox <gi...@apache.org>.
F21 merged pull request #144:
URL: https://github.com/apache/calcite-avatica/pull/144


   


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

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

[GitHub] [calcite-avatica] julianhyde commented on pull request #144: [CALCITE-4575] Remove Gradle wrapper from source distribution

Posted by GitBox <gi...@apache.org>.
julianhyde commented on pull request #144:
URL: https://github.com/apache/calcite-avatica/pull/144#issuecomment-839419870


   I've pushed it to the same branch. No PR needed.


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

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

[GitHub] [calcite-avatica] F21 commented on pull request #144: [CALCITE-4575] Remove Gradle wrapper from source distribution

Posted by GitBox <gi...@apache.org>.
F21 commented on pull request #144:
URL: https://github.com/apache/calcite-avatica/pull/144#issuecomment-839360838


   Was the fix for CALCITE-4576 a separate commit? I think you forgot to push it, so you will need to open a new PR for it.


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

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