Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9440 )
Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/9440/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/9440/1//COMMIT_MSG@10 PS1, Line 10: orders by selectivity (as before), but breaks ties by column index. > do you think determinism should be independent of the order in which the pr Pretty sure we don't maintain the order, eg https://github.com/apache/kudu/blob/master/src/kudu/common/scan_spec.h#L146. Eventually I think we should be taking into account tablet and perhaps DRS metadata to make this decision, so I hesitate to make any guarantees about pass-thru. -- To view, visit http://gerrit.cloudera.org:8080/9440 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I99b2cabecd8626cad7e11fbdd492af7276e08348 Gerrit-Change-Number: 9440 Gerrit-PatchSet: 1 Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Comment-Date: Sat, 24 Feb 2018 01:35:25 +0000 Gerrit-HasComments: Yes
