binarycat0 opened a new issue, #2488:
URL: https://github.com/apache/polaris/issues/2488

   Thanks for the perf. update @binarycat0 ! However, the execution times are 
too small and the sample size is small too (I'm afraid) so the perf. numbers 
are not decisive, IMHO :sweat_smile: 
   
   At the same time the new index will have some storage and maintenance 
overhead.
   
   Let's open a `dev` ML discussion about this and see what people who use 
Polaris with Postgres think.
   
   The java code changes LGTM :+1:
   
   _Originally posted by @dimas-b in 
https://github.com/apache/polaris/issues/2465#issuecomment-3245776381_
               


-- 
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: issues-unsubscr...@polaris.apache.org.apache.org

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

Reply via email to