OpenMLDB icon indicating copy to clipboard operation
OpenMLDB copied to clipboard

window project over limit clause is not supported

Open aceforeverd opened this issue 3 years ago • 1 comments

since fdfd1f29c6b0ed46a822afdd459ea17f567ccd66, window project over limit clause is not supported. Because LimitTableHandler is not seekable: https://github.com/4paradigm/OpenMLDB/blob/5ac4667d0e71dc803ddbd04928bc4681b5ba84b6/hybridse/src/vm/catalog_wrapper.h#L444-L448

This kind SQL might not correct.

select id, count(col) over w from (select * from t1 limit 10) window w as (....)

aceforeverd avatar Sep 09 '22 08:09 aceforeverd

wrongest solution: LimitTableHandler is used for the lazyness of execution. When window iterator is need, a full copy of limited rows are copyed into new TableHandler.

aceforeverd avatar Sep 09 '22 08:09 aceforeverd

same for

  • window (filter)
  • join (filter)

aceforeverd avatar Mar 15 '23 14:03 aceforeverd