Valery Mironov
Valery Mironov
@mbasmanova PR description is updated, commits are rebased and squashed
@mbasmanova I made update, to resolve some of your comments, see https://github.com/facebookincubator/velox/pull/15734/commits/6e44a9901b8929ebdc3e916f45362e802590036c Mainly it's about comments and removing `Cost.h` and `velox::Cost` For other comments I answered. What is rest? 1....
@mbasmanova I made separate refactoring PR and rebased this PR on refactoring PR, see https://github.com/facebookincubator/velox/pull/15769 I hope it's ok, if not, please feel free to get this changes in upstream...
@bikramSingh91 Thanks for looking this over. I see your goal of wanting maximum clarity in the changes. For this contribution, we already structured it into the two parts: 1) refactoring...
I think PR description is outdated, because other PRs with same purpose was merged first, although created later. This is sad. Anyway this is updated version of PR with more...
Don't forget to use "Hide whitespace" feature
A lot of conflicts was introduced, so I close it