{"push_id":2971248809,"size":1000,"distinct_size":244,"ref":"refs/heads/cdh5-trunk","head":"3b6a4f8f5c365168a7bf3687757907f0f075a7d2","before":"40c01a7f92d2248229e8e45291a1ef43b8c40f48","commits":[{"sha":"89091f602660ad258617b0eafc786472bc24c6f1","author":{"name":"John Russell","email":"308573a947c636ab812b35943b6242633bef8113@cloudera.com"},"message":"[DOCS] Update list of reserved words\n\nFor 2.9, I believe the only new reserved\nkeyword is TABLESAMPLE from IMPALA-5309.\n\nBased on commit history from:\nhttps://github.com/apache/incubator-impala/commits/master/fe/src/main/jflex/sql-scanner.flex\n\nChange-Id: If4a340a033ff3f529061e48c4a5558b1ad1637ef\nReviewed-on: http://gerrit.cloudera.org:8080/7452\nReviewed-by: Michael Brown <mikeb@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/89091f602660ad258617b0eafc786472bc24c6f1"},{"sha":"bed85e1ff3afcfac0fcefd6605473866ed279d54","author":{"name":"Matthew Jacobs","email":"df2fa7c39d141dc5dc5589fed74180e757ed6c9a@cloudera.com"},"message":"Bump Kudu version to 27854fd\n\nChange-Id: I0b9414c21faca00e4a64a35888bd50caac94318f\nReviewed-on: http://gerrit.cloudera.org:8080/7486\nReviewed-by: Thomas Tauber-Marshall <tmarshall@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/bed85e1ff3afcfac0fcefd6605473866ed279d54"},{"sha":"f8491de02ae1dc58941205bde6eb8a88b76d7bd5","author":{"name":"John Russell","email":"308573a947c636ab812b35943b6242633bef8113@cloudera.com"},"message":"[DOCS] Take out not-production-ready notice from ADLS page\n\nChange-Id: I127b7806feca810503ba3dd000a8e972835e715a\nReviewed-on: http://gerrit.cloudera.org:8080/7487\nReviewed-by: Greg Rahn <grahn@cloudera.com>\nReviewed-by: Sailesh Mukil <sailesh@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/f8491de02ae1dc58941205bde6eb8a88b76d7bd5"},{"sha":"15d05abed6e8190a56d11e9765d51d223c2eb3b4","author":{"name":"Henry Robinson","email":"3eca10f30041813f045165784e24b5a950a6cc7e@cloudera.com"},"message":"IMPALA-4905: Don't send empty insert status to coordinator\n\nIf the coordinator, in UpdateBackendExecStatus(), receives a report that\nincludes a TInsertExecStatus, it will call UpdateInsertExecStatus()\nwhich takes the coordinator-wide lock. Avoid doing this for fragment\ninstances that would only send an empty TInsertExecStatus (including\ninstances that belong to SELECT queries, not DML queries).\n\nFuture changes should fix the locking around the\nUpdateBackendExecStatus() path to remove dependencies on\nCoordinator::lock_, but this fix is simple and addresses one point of\nneedless contention.\n\nChange-Id: I314dd8d96922d273c6329266970d249ec8c5c608\nReviewed-on: http://gerrit.cloudera.org:8080/7457\nReviewed-by: Henry Robinson <henry@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/15d05abed6e8190a56d11e9765d51d223c2eb3b4"},{"sha":"ac69517addfc7249422ba4184d51369157d83b2f","author":{"name":"Henry Robinson","email":"3eca10f30041813f045165784e24b5a950a6cc7e@cloudera.com"},"message":"IMPALA-5709: Remove mini-impala-cluster\n\nRemove untested / unused mini-impala-cluster binary.\n\nChange-Id: I677314fc1a998dffa9120c016bfcf761b4e39f05\nReviewed-on: http://gerrit.cloudera.org:8080/7488\nReviewed-by: Matthew Jacobs <mj@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/ac69517addfc7249422ba4184d51369157d83b2f"},{"sha":"aaac95e04dc7509d7446ded3d07edafc4ff9cb48","author":{"name":"Bharath Vissapragada","email":"f23dbca75acdac4082d6a05c290fc03565dada86@cloudera.com"},"message":"IMPALA-5615: Fix compute incremental stats for general partition exprs\n\nThe fix for IMPALA-1654 has broken the compute incremental stats child\nquery generation logic for general partition expressions. This commit\nfixes it and also adds new queries to fix the test gap. These tests\nfail consistently without the patch.\n\nChange-Id: I227fc06f580eb9174f60ad0f515a3641cec19268\nReviewed-on: http://gerrit.cloudera.org:8080/7379\nReviewed-by: Bharath Vissapragada <bharathv@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/aaac95e04dc7509d7446ded3d07edafc4ff9cb48"},{"sha":"e862385281c74baa6f1a4d10d44f3411a4303abc","author":{"name":"aphadke","email":"b1ca4da392c8ba945e90771bee93294a6925172b@cloudera.com"},"message":"IMPALA-5586: Null-aware anti-join can take a long time to cancel\n\nQueries with a null-aware anti-join joining on a large number of NULLs\ncan take a long time to cancel if threads are stuck in\nPartitionedHashJoinNode::EvaluateNullProbe(). This change adds the\nRETURN_IF_CANCELLED macro to the function.\n\nTesting:\nAdded logs to PartitionedHashJoinNode::EvaluateNullProbe() and made sure\nthat the function returns right away on cancellation.\n\nChange-Id: I0800754d4ad31cbadbdfadc630c640963f3f6053\nReviewed-on: http://gerrit.cloudera.org:8080/7393\nTested-by: Impala Public Jenkins\nReviewed-by: Tim Armstrong <tarmstrong@cloudera.com>","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/e862385281c74baa6f1a4d10d44f3411a4303abc"},{"sha":"b8f4391d2cff51f869e8fd8e40d9e177ca3c0878","author":{"name":"Matthew Jacobs","email":"df2fa7c39d141dc5dc5589fed74180e757ed6c9a@cloudera.com"},"message":"IMPALA-5489: Improve Sentry authorization for Kudu tables\n\nIMPALA-4000 added basic authorization support for Kudu\ntables, but it had several limitations:\n* Only the ALL privilege level can be granted to Kudu tables.\n  (Finer-grained levels such as only SELECT or only INSERT are\n  not supported.)\n* Column level permissions on Kudu tables are not supported.\n* Only users with ALL privileges on SERVER may create external\n  Kudu tables.\n\nThis patch relaxes the restrictions to allow:\n* Allow column-level permissions\n* Allow fine grained privileges SELECT and INSERT for those\n  statement types.\n\nDELETE/UPDATE/UPSERT privileges now require ALL privileges\nbecause Sentry will eventually get fine grained privilege\nactions, and at that point Impala should support the more\nspecific actions (IMPALA-3840). The assumption is that the\nKudu table authorization support is currently so limited\nthat most users are not using this functionality yet, but\nthis is a behavior change that needs to be clearly stated in\nthe Impala release notes.\n\nTesting: Adds FE and EE tests.\n\nChange-Id: Ib12d2b32fa3e142e69bd8b0f24f53f9e5cbf7460\nReviewed-on: http://gerrit.cloudera.org:8080/7307\nReviewed-by: Matthew Jacobs <mj@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/b8f4391d2cff51f869e8fd8e40d9e177ca3c0878"},{"sha":"aa7e38395fbf3fb4c871ec022827a9e47c92d0ab","author":{"name":"Michael Brown","email":"97e4d5d0e400e5ad37382425ea95bde8b3933403@cloudera.com"},"message":"IMPALA-3496: stress test: print version info\n\nPrint the version info of each impalad that's used in a stress test run,\nsorted by host name.\n\nTesting done:\n$ tests/stress/concurrent_select.py [redacted cluster options] --tpcds-db null --max-queries 0\nCluster Impalad Version Info:\nhost2.redacted: impalad version 2.10.0-SNAPSHOT RELEASE (build e862385281c74baa6f1a4d10d44f3411a4303abc)\nBuilt on Tue Jul 25 07:06:27 PDT 2017\nhost3.redacted: impalad version 2.10.0-SNAPSHOT RELEASE (build e862385281c74baa6f1a4d10d44f3411a4303abc)\nBuilt on Tue Jul 25 07:06:27 PDT 2017\nhost4.redacted: impalad version 2.10.0-SNAPSHOT RELEASE (build e862385281c74baa6f1a4d10d44f3411a4303abc)\nBuilt on Tue Jul 25 07:06:27 PDT 2017\nhost5.redacted: impalad version 2.10.0-SNAPSHOT RELEASE (build e862385281c74baa6f1a4d10d44f3411a4303abc)\nBuilt on Tue Jul 25 07:06:27 PDT 2017\nhost6.redacted: impalad version 2.10.0-SNAPSHOT RELEASE (build e862385281c74baa6f1a4d10d44f3411a4303abc)\nBuilt on Tue Jul 25 07:06:27 PDT 2017\n2017-07-25 12:38:52,732 12793 Thread-1 INFO:cluster[691]:Finding impalad binary location\n...\n\nChange-Id: Ie4b40783ddae6b1bfb2bb4e28c0e3bf97ab944c5\nReviewed-on: http://gerrit.cloudera.org:8080/7501\nReviewed-by: Michael Brown <mikeb@cloudera.com>\nTested-by: Michael Brown <mikeb@cloudera.com>","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/aa7e38395fbf3fb4c871ec022827a9e47c92d0ab"},{"sha":"c5d80aca2cf7d20fbebaa9c80dc6dd7c16804aa5","author":{"name":"Gabor Kaszab","email":"f3d16c4d12a518b26feeeda5cb56450f7b6c1f3b@cloudera.com"},"message":"IMPALA-5511: Add process start time to debug web page\n\nRead the start date and time of the impalad, catalogd and statestored processes\nfor the Debug Web UI. Uses the stat command on the /proc/<pid> directory and\nformat the date with the date command to local time format.\n\nChange-Id: I05ae2f80835b1b0e4bc3b38731778ba0871338a4\nReviewed-on: http://gerrit.cloudera.org:8080/7363\nReviewed-by: Matthew Jacobs <mj@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/c5d80aca2cf7d20fbebaa9c80dc6dd7c16804aa5"},{"sha":"8e446c33539429fcfa3d7e161485c2d9cb6d5833","author":{"name":"Tim Armstrong","email":"2f8e34250f03a653da3dc27f7ec203eafdde80fc@cloudera.com"},"message":"IMPALA-5691: recalibrate mem limit for Q18\n\nI ran the stress test binary search locally and it produced a slightly\nhigher number for Q18 than the hardcoded value. This is enough to move\nit above one of the thresholds, so may reduce flakiness.\n\nTesting:\nI wasn't able to reproduce the flakiness locally, so can't confirm\nthis fixes it.\n\nChange-Id: I1ffa969061a52730c5147d142dcd2e3cb3626590\nReviewed-on: http://gerrit.cloudera.org:8080/7512\nReviewed-by: Matthew Jacobs <mj@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/8e446c33539429fcfa3d7e161485c2d9cb6d5833"},{"sha":"178a6a186ae0d156cc51ed6ef4547ec1d9618000","author":{"name":"Henry Robinson","email":"3eca10f30041813f045165784e24b5a950a6cc7e@cloudera.com"},"message":"IMPALA-5716: Don't delete cmake_modules/* when enabling distcc\n\nIf $IMPALA_HOME ends with a /, the clean_cmake_files function in\ndistcc_env.sh will emit a find command with a double // at the end for\nthe cmake_modules directory, and since it contains the substring cmake,\nfind will match and delete its contents.\n\nFix is to use a whitelist of locations and filenames to look for, and\ndelete only those.\n\nTesting: manually ran enable_distcc, observed that my files were still\nthere.\n\nChange-Id: I8a6e34bedf8000aed9e2b0597cfe86f73222c6ed\nReviewed-on: http://gerrit.cloudera.org:8080/7493\nReviewed-by: Tim Armstrong <tarmstrong@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/178a6a186ae0d156cc51ed6ef4547ec1d9618000"},{"sha":"0a99105856610cc6d498ea3835e6e6a9060543e3","author":{"name":"Taras Bobrovytsky","email":"72e4a6fc4c5ef23f318dc209e8402725249a5676@cloudera.com"},"message":"IMPALA-5689: Avoid inverting non-equi left joins\n\nWhen checking if a join can be inverted, we forgot to also check that\nthe resulting join would not be a non-equi right semi-join or a\nnon-equi right outer-join. We currently do not support those kinds of\njoins in the backend.\n\nTesting:\n-Added a planner test\n\nChange-Id: I91ba66fe30139fcd44d4615a142f183266800aab\nReviewed-on: http://gerrit.cloudera.org:8080/7476\nReviewed-by: Taras Bobrovytsky <tbobrovytsky@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/0a99105856610cc6d498ea3835e6e6a9060543e3"},{"sha":"a24310aa1ac89e397d35568984f2f38df631af45","author":{"name":"Hector Acosta","email":"a536529b3b13147e1c7ec69edb96289c6332fc8b@cloudera.com"},"message":"IMPALA-5739: Correctly handle sles12 SP2\n\nThis takes care of the difference in outputs for SLES 12 SP1 and SP2.\nFor reference here's the outputs in sles12sp1 and sp2:\n\nsles12sp1 # lsb_release -irs\nSUSE LINUX 12.1\nsles12sp2 # lsb_release -irs\nSUSE 12.2\n\nTesting:\nDid a full build on SLES12 SP2. Before this patch, a build resulted in:\n'Pre-built toolchain archives not available for your platform.'\n\nAfter this patch:\nToolchain bootstrap complete.\n\n..Followed by a full build.\n\nChange-Id: I005e05b8b66de78e6d53a35a894eb34d89843a62\nReviewed-on: http://gerrit.cloudera.org:8080/7535\nTested-by: Tim Armstrong <tarmstrong@cloudera.com>\nReviewed-by: Tim Armstrong <tarmstrong@cloudera.com>","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/a24310aa1ac89e397d35568984f2f38df631af45"},{"sha":"da059d7bc3e025e11d0157a2f5bcc46fba500d0d","author":{"name":"Henry Robinson","email":"3eca10f30041813f045165784e24b5a950a6cc7e@cloudera.com"},"message":"IMPALA-4669: [KUTIL] Add kudu_util library to the build.\n\nA few miscellaneous changes to allow kudu_util to compile with Impala.\n\nAdd kudu_version.cc to substitute for the version.cc file that is\nautomatically built during the full Kudu build.\n\nSet LZ4_DISABLE_DEPRECATE_WARNINGS to allow Kudu's compressor utility to\nuse deprecated names for LZ4 methods.\n\nAdd NO_NVM_SUPPORT flag to Kudu build (plan to upstream this later) to\ndisable building with nvm support, removing a library dependency.\n\nAlso remove imported FindOpenSSL.cmake in favour of the standard one provided\nby cmake itself.\n\nFinally, a few changes to allow compilation on RHEL5:\n\n* Only use sched_getcpu() if supported\n* Only include magic.h if available\n* Workaround for kernels that don't have SOCK_NONBLOCK\n* Workaround for kernels that don't have O_CLOEXEC (ignore the flag)\n* Provide non-working implementation of fallocate()\n* Disable inclusion of linux/fiemap.h - although this exists on RHEL5,\n  it does not compile due to other #includes in env_posix.cc. We disable\n  the path this is used for, since Impala does not call that code.\n* Use Kudu's implementation of pipe(2), preadv(2) and pwritev(2) where\n  it doesn't exist.\n\nIn most cases these changes simply force kutil to revert to a different\nimplementation that was already written for OSX support - this patch\ngeneralises the logic to provide the implementation whenever the\nrequired function doesn't exist.\n\nThis patch compiles on RHEL5.5 and 6.0, SLES11 and 12, Ubuntu 12.04 and\n14.04 and Debian 7.0 and 8.0.\n\nChange-Id: I8da798ee55506d6f969416b17c191eb03cb215f5\nReviewed-on: http://gerrit.cloudera.org:8080/5715\nTested-by: Impala Public Jenkins\nReviewed-by: Henry Robinson <henry@cloudera.com>","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/da059d7bc3e025e11d0157a2f5bcc46fba500d0d"},{"sha":"a0fe123e2976f100ccbe3a0244a604318f29a5c4","author":{"name":"Michael Ho","email":"43544f6c6f19372b140301f92a3768afc300868a@cloudera.com"},"message":"IMPALA-5714: Add OpenSSL to bootstrap_toolchain.py\n\nTo support KRPC on legacy platforms with version of OpenSSL\nolder than 1.0.1, we may need to use libssl from the toolchain.\nThis change makes toolchain boostrapping to also download\nOpenSSL 1.0.1p.\n\nTesting: private packaging build.\n\nChange-Id: I860b16d8606de1ee472db35a4d8d4e97b57b67ae\nReviewed-on: http://gerrit.cloudera.org:8080/7532\nReviewed-by: Henry Robinson <henry@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/a0fe123e2976f100ccbe3a0244a604318f29a5c4"},{"sha":"f58f55f6b014c996d6dd4d6a7f26794e14a42c1f","author":{"name":"Lars Volker","email":"4d73c29a4561048ce85fa0051bb7a6d201781159@cloudera.com"},"message":"IMPALA-4086: Add benchmark for simple scheduler\n\nChange-Id: I89ec1c6c1828bb0b86d1e13ce4dfc5a8df865c2e\nReviewed-on: http://gerrit.cloudera.org:8080/4554\nReviewed-by: Lars Volker <lv@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/f58f55f6b014c996d6dd4d6a7f26794e14a42c1f"},{"sha":"73ac9e6f3a941c7d8f1a733b9edc8210a408c95d","author":{"name":"Taras Bobrovytsky","email":"72e4a6fc4c5ef23f318dc209e8402725249a5676@cloudera.com"},"message":"IMPALA-5722: Fix string to decimal cast\n\nWhen converting a string to a decimal, we didn't handle the case where\nthe exponent is a large negative number. The function for computing the\ndivisor returns -1 when the exponent is too large. The problem is fixed\nby making sure that the divisor is positive.\n\nTesting:\n-Added decimal tests.\n\nChange-Id: I1f5eb5b64a6924af2e8eb7f9a50da67015757efe\nReviewed-on: http://gerrit.cloudera.org:8080/7517\nReviewed-by: Tim Armstrong <tarmstrong@cloudera.com>\nReviewed-by: Jim Apple <jbapple-impala@apache.org>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/73ac9e6f3a941c7d8f1a733b9edc8210a408c95d"},{"sha":"b8148afa4657223fdaa5cac76248848ccfc642a1","author":{"name":"Bikramjeet Vig","email":"f534d302728ba886c91adfa78952ab96e2b2b265@cloudera.com"},"message":"IMPALA-1882: Remove ORDER BY restriction from first_value()/last_value()\n\nIn order to conform to the ISO SQL specifications, this patch allows\nthe 'order by' clause to be optional for first_value() and last_value()\nanalytical functions.\n\nTesting:\n1. Added Analyzer tests\n2. Added Planner tests which checks that a sort node is not used if both\n'order by' and 'partition by' clauses are not used. Also, checks that\nif only 'partition by' clause is used then the sorting is done only on\nthe partition column.\n3. Added a query test that checks that the input is not reordered by\nthese analytic functions if 'order by' clause is not used\n\nChange-Id: I5a3a56833ac062839629353ea240b361bc727d96\nReviewed-on: http://gerrit.cloudera.org:8080/7502\nReviewed-by: Matthew Jacobs <mj@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/b8148afa4657223fdaa5cac76248848ccfc642a1"},{"sha":"ed6aedaa4c4ddba121be1194f2824517ed3f9acf","author":{"name":"Bikramjeet Vig","email":"f534d302728ba886c91adfa78952ab96e2b2b265@cloudera.com"},"message":"IMPALA-5630: Add Kudu client version as a common metric\n\nChange-Id: I8bd0e220c030822209b811703ebd0cea699e8268\nReviewed-on: http://gerrit.cloudera.org:8080/7525\nReviewed-by: Matthew Jacobs <mj@cloudera.com>\nTested-by: Impala Public Jenkins","distinct":false,"url":"https://api.github.com/repos/cloudera/Impala/commits/ed6aedaa4c4ddba121be1194f2824517ed3f9acf"}]}
{"actor":{"display_login":"cloudera-jenkins"}}