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 2022/10/22 08:08:47 UTC

[GitHub] [calcite] asolimando commented on pull request #2947: [MINOR] Add default catalog link to test fixtures

asolimando commented on PR #2947:
URL: https://github.com/apache/calcite/pull/2947#issuecomment-1287672624

   I have recently had the same issue, so +1 on the change.
   
   I noticed that the comments for the `sql()` method are all different for these three classes, maybe we can try to provide a single description, so that it's clear that the three methods are doing the same thing?


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