Walaa Eldin Moustafa

> A UDF is considered shaded if the prefix before the first dot contains an underscore. This convention is brittle. Why is shading indicated by an underscore? Can we use...

> If i understand correctly @wmoustafa comment on the mailing list, then there is some ambiguity here for what to put, if the same table in expressed in the various...

> > > If i understand correctly @wmoustafa comment on the mailing list, then there is some ambiguity here for what to put, if the same table in expressed in...

I think overall, the PR could benefit from more extensive documentation. Current documentation on different methods is a good start. Each documentation should be expanded with concrete running example with...

Overall feedback is that I believe the PR sounds too specific and there is no general framework for cost estimation that is applicable to various operators by specializing some of...

Thanks! There is Coral-service which deploys Coral as an HTTP service and users can interact with it from the browser. There is no packaged docker image yet, and the team...

> There are two mechanisms in v3 for building abstract syntax trees (ASTs): operators and rewrite rules. > -> Using the antlr3 rewrite symbol ^ tree construction operator Could you...

While it is a good catch, I am not sure if it is sufficient to delete it based on the current tests passing. We probably need to dig deeper to...

> In `GenericProjectTransformer` we compare ... Can we always introduce which part of the pipeline this happens? Example: In LHS, when converting from Hive SqlNode to Coral SqlNode, etc. Heads...

> As discussed offline, please check if it is possible to use StructKind.PEEK_FIELDS_NO_EXPAND during all RelNode creation steps What are the "all RelNode creation steps"? How to identify them?