vlsi edited a comment on issue #1792: [CALCITE-3775] Implicit lookup methods in 
SimpleCalciteSchema ignore case sensitivity parameter
URL: https://github.com/apache/calcite/pull/1792#issuecomment-584757275
 
 
   >But the Schema interface is important to end-users. Modifying the interface 
may bring impact on compatibility.
   
   There are ways to do that without breaking compatibility much:
   a) Release Calcite 2.0
   b) Add a method with `default` implementation
   
   Both of the options are better than adding a half-baked solution to 
SimpleCalciteSchema.
   
   > SimpleCalciteSchema needs to **construct the data structure** each time.
   
   I do not see that documented.
   Can you please pin-point?
   

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to