sunjincheng121 edited a comment on pull request #2600:
URL: https://github.com/apache/calcite/pull/2600#issuecomment-963780306


   > @julianhyde , Here's the discussion: 
https://lists.apache.org/thread/zyrftnms4p31lsvm93x1cynhnjnccbq7
   > 
   > So far it looks like I broke both calcite-avatica and calcite builds by 
merging 
[apache/calcite-avatica#161](https://github.com/apache/calcite-avatica/pull/161).
 That is why I try to keep the thing going and try to figure out to heal both 
builds without reverting PR#161.
   > 
   > As an alternative option, I could revert PR#161 and let somebody else deal 
with the situation.
   
   Thanks for helping review and merged the PR161. and broke the builds is my 
fault. and de-coupling the project's master dependency of the two projects is 
what we want to see.  Will Pay attention to the 
[discussion](https://lists.apache.org/thread/zyrftnms4p31lsvm93x1cynhnjnccbq7) 
you initiated before making any changes. (I'll subscribe the 
[email protected] today later)


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


Reply via email to