HappenLee commented on pull request #6866: URL: https://github.com/apache/incubator-doris/pull/6866#issuecomment-954493740
> Is the Reader interface necessary? I think the V**Node is a separate interface. Maybe we only need BlockReader and TupleReader? Yes, tuple_reader and blockr_eader have many public methods that can be reused on the reader, so the reader is retained as the parent class -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org