Hello Jim Apple, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8355 to look at the new patch set (#8). Change subject: IMPALA-5754: Improve randomness of rand()/random() ...................................................................... IMPALA-5754: Improve randomness of rand()/random() Currently implementation of rand/random built-in functions use rand_r of C library. We recognized its randomness was poor. std::mt19937 in the C++11 standard libarary shows better randomness than rand_r. Testing: Revise unit test in expr-test Add E2E test to random.test Change-Id: Idafdd5fe7502ff242c76a91a815c565146108684 --- M be/src/exprs/expr-test.cc M be/src/exprs/math-functions-ir.cc A testdata/workloads/functional-query/queries/QueryTest/random.test M tests/query_test/test_queries.py 4 files changed, 61 insertions(+), 29 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/55/8355/8 -- To view, visit http://gerrit.cloudera.org:8080/8355 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Idafdd5fe7502ff242c76a91a815c565146108684 Gerrit-Change-Number: 8355 Gerrit-PatchSet: 8 Gerrit-Owner: Kim Jin Chul <jinc...@gmail.com> Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org> Gerrit-Reviewer: Kim Jin Chul <jinc...@gmail.com>