[GitHub] [calcite] sunjincheng121 edited a comment on pull request #2600: [CALCITE-4877] Make the exception information more explicit for instantiate plugin.

[GitBox <gi...@apache.org>]

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 dev@calcite.apache.org 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: commits-unsubscribe@calcite.apache.org

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