Zihao Xu

Results 18 comments of Zihao Xu

> Can you elaborate on the relationship between this PR and #14965? #14965 aims to optimize the pattern like below, e.g., ``` select case 1 when 1 then 114514 (...)...

P.S. It's strange that only in `StreamFilter` (presumably in `LogicalShare`) will these kind of predicate exist, so the current implementation is essentially checking the input in `LogicalShare`, I have no...

> We have had some expressions simplify logic here and it could be called every time a new `Condition` is constructed. > > https://github.com/risingwavelabs/risingwave/blob/d9228fcb4681206813ea7ea943991d33c69a1f5d/src/frontend/src/utils/condition.rs#L845 > > > > I am...

The essential problem for expression simplification in general at present, is that we can't ensure **whether to transform** the underlying expression to if `Strong::is_null` returns false. As an example, for...

Unfortunately according to the second case, my previous imagination is actually wrong, i.e., > As an general extension, for expression like Not(c1 > 1 and c2 > 2 and ......

Now we only consider `e` that contains at most one column as the optimizable pattern, in `Not(e) [op] e`, if the column exists, it will be converted to `IsNotNull(col)` under...

Current dependencies on/for this PR: * **#15188** 👈 * `main` This [stack of pull requests](https://stacking.dev/?utm_source=stack-comment) is managed by [Graphite](https://app.graphite.dev/github/pr/risingwavelabs/risingwave/15188?utm_source=stack-comment).

Also, I think the asynchronous version of the four functions `WritePage`, `ReadPage`, `WriteLog`, `ReadLog` can be of help if `bustub` future will consider in providing `WAL` related functionalities..🤤 (Actually I...

I revisited the original code design and found it confusing for logging operations, so I redesigned some of the `functions` & `variables` related to `Log`. 1. In the original design,...

This part may also be associated with `LogManager`. If anything related to `LogManager` or the currently disabled recovery test should also be changed, I'm glad to help with the refactoring...