Abhishek Chennaka has uploaded this change for review. ( http://gerrit.cloudera.org:8080/21769
Change subject: Squash cherry-picked commits #6 This is a combination of 20 commits. ...................................................................... Squash cherry-picked commits #6 This is a combination of 20 commits. This is the 1st commit message: [rpc] increase listened socket backlog up to 512 This patch updates the default value for --rpc_acceptor_listen_backlog, setting it up to 512 by default. This is to help busy Kudu servers accommodating of larger bursts of incoming requests for new RPC connections. In addition, a warning message is output into the log if the setting for the flag is effectively capped by the system-level limit. I manually verified that the warning is issued as expected on Linux and macOS, correspondingly: --rpc_acceptor_listen_backlog setting 512 is capped at 128 by /proc/sys/net/core/somaxconn --rpc_acceptor_listen_backlog setting 512 is capped at 256 by kern.ipc.somaxconn Reviewed-on: http://gerrit.cloudera.org:8080/20797 Tested-by: Kudu Jenkins Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 6c02274ce275615fbbc83703b6f695a0a53c87f1) This is the commit message #2: [tests] fix flakiness in TestSpaceAvailableMetrics I saw TabletServerDiskErrorITest.TestSpaceAvailableMetrics failing with an error like below in a pre-commit test run [1] (TSAN build): src/kudu/integration-tests/disk_failure-itest.cc:352: Failure Expected equality of these values: wal_dir_space Which is: 40835236209 wal_dir_space_refetch Which is: 40767713649 This patch addresses the issue. [1] http://dist-test.cloudera.org/job?job_id=jenkins-slave.1703028347.1027630 Reviewed-on: http://gerrit.cloudera.org:8080/20821 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit dc64df6c1219dc2c4e930f6ea8d5106d3c7fb5cf) This is the commit message #3: [Refactor] Fix a memory leak warning The following is the warning messege: src/kudu/consensus/quorum_util-test.cc:104:1: warning: Potential leak of memory pointed to by 'attrs_pb._M_t._M_t._M_head_impl' [clang-analyzer-cplusplus.NewDeleteLeaks] src/kudu/consensus/quorum_util-test.cc:1822:3: note: Calling 'AddPeer' AddPeer(&config, "A", V, '+', {{"REPLACE", true}}); src/kudu/consensus/quorum_util-test.cc:85:3: note: Taking true branch if (overall_health) { src/kudu/consensus/quorum_util-test.cc:90:7: note: Assuming the condition is true if (!attrs.empty()) { src/kudu/consensus/quorum_util-test.cc:90:3: note: Taking true branch if (!attrs.empty()) { src/kudu/consensus/quorum_util-test.cc:91:42: note: Memory is allocated unique_ptr<RaftPeerAttrsPB> attrs_pb(new RaftPeerAttrsPB); src/kudu/consensus/quorum_util-test.cc:104:1: note: Potential leak of memory pointed to by 'attrs_pb._M_t._M_t._M_head_impl' Reviewed-on: http://gerrit.cloudera.org:8080/20784 Reviewed-by: Yingchun Lai <[email protected]> Tested-by: Yingchun Lai <[email protected]> (cherry picked from commit 2a9250bb21c83d0522a5e1ac144e7ac15fcbed01) This is the commit message #4: [util] Add a warning into 'heap_sample_every_n_bytes' flag description. Parameter --heap_sample_every_n_bytes is actually a parameter of tcmalloc. Setting it to a positive value will make thread cache request some sample allocations from page cache, which will compete the lock of page heap with other threads. So does the free operation of the sample. A tserver with high throughput might lose so much performance because of the lock contention of tcmalloc. Almost all the apply threads are waiting for the spin lock. So add a warning on --heap_sample_every_n_bytes in case that some users do not know the terrible influence of this parameter. Reviewed-on: http://gerrit.cloudera.org:8080/20716 Tested-by: Kudu Jenkins Reviewed-by: Wang Xixu <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit 84c838d3b687c498467c87507eec687c446347c7) This is the commit message #5: [webserver] add security-related HTTP headers To please various security scanners, this patch adds the following HTTP headers into Kudu embedded webserver's responses: * Cache-Control (set to 'no-store' by default, see [1]) * X-Content-Type-Options (set to 'nosniff' by default, see [2]) * Strict-Transport-Security (see [3] and below for details) The embedded webserver adds the HTTP strict transport security (HSTS) header 'Strict-Transport-Security' for responses sent from HTTPS (i.e. TLS-protected) endpoints if --webserver_hsts_max_age_seconds is set to a non-negative value. The header contains the 'max-age' attribute as specified by the flag, and adds the optional 'includeSubDomains' attribute as per set setting of the --webserver_hsts_include_sub_domains flag. The HSTS header isn't added to the responses sent from plain HTTP endpoints (BTW, it seems most browsers simply ignore the HSTS header anyway if it's received from an HTTP, not an HTTPS endpoint). Essentially, the HSTS header for Kudu is a no-op since the embedded webserver doesn't serve both HTTP and HTTPS endpoints at the same time: one can enable one or the other, but never both. However, many security scanners almost cry "Security breach" if they don't see the header :) Adding the HSTS header isn't enabled by default since it could make other plain HTTP endpoints at the same node/hostname inaccessible. To enable adding the HSTS header for HTTPS responses, set the --webserver_hsts_max_age_seconds flag to a non-negative integer. Enable it with care and only if you know what you are doing! One extra test added and a few existing ones updated correspondingly to cover the newly introduced functionality. [1] https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cache-Control [2] https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Content-Type-Options [3] https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Strict-Transport-Security Reviewed-on: http://gerrit.cloudera.org:8080/18253 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Attila Bukor <[email protected]> (cherry picked from commit 26ad5ad962fe05af4b69730ffded9414fd13a5e5) This is the commit message #6: KUDU-3534 [compaction] Log timestamp of two matching DELETE REDO mutations. This patch just adds an info message to log timestamp value in case two REDO mutations of type DELETE are found to be stamped with same time. This is an undesired condition that could possibly happen due to corruption of mutation entries. The value will give us an idea whether it is 0, garbled or close to some valid timestamp. Reviewed-on: http://gerrit.cloudera.org:8080/20792 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Abhishek Chennaka <[email protected]> Reviewed-by: Wang Xixu <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit 1f6f7e5bafb6a9f497ea92b62c1f268a2fe6c3c3) This is the commit message #7: [rpc] extra test scenarios for exactly-once RPC logic In the context of troubleshooting one issue, it was necessary to know how the exactly-once RPC logic handles the re-ordering of RPC requests, so I added these extra scenarios to make sure the implementation behaves as expected. I think it's worth it adding the new tests into the code base to explicitly state how the system behaves in particular cases and to catch possible regressions if the corresponding code modified in the future. Reviewed-on: http://gerrit.cloudera.org:8080/20831 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 0c8d2eee1ee2b03d0de369d5975f153b00e26b36) This is the commit message #8: Do not enter ALTERING state just for replication factor changed. If we change the replication factor of a table without any tablets, the table will stay in ALTERING state since exiting this state needs tablet reports from the tablets of this table. for now the condition of entering ALTERING state is as below: has_metadata_changes && (table->num_tablets() > tablets_to_drop.size() || num_replicas_changed); Actually even if the number of replicas has been changed, the former condition is also needed. So rollback the code to: has_metadata_changes && table->num_tablets() > tablets_to_drop.size(); Reviewed-on: http://gerrit.cloudera.org:8080/20830 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit 7b1e179915a32b7c338025eae6225005befc9f43) This is the commit message #9: [java] add sequenceId into the tracked RPCs' trace This patch adds sequenceId into the result of KuduRpc.ToString() for tracked RPCs (i.e. Write RPC as of now). That's to provide better tracking of retries performed in the context of exactly-once semantics. I found this information would be helpful when troubleshooting particular scenarios. Otherwise, there are no functional modifications in this changelist. Reviewed-on: http://gerrit.cloudera.org:8080/20857 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Kudu Jenkins (cherry picked from commit b88e3e9e46aaa710d7c32558a1aa355d0bee7913) This is the commit message #10: [log] use THROTTLE_MSG for KLOG_EVERY_N_SECS when necessary As a follow-up to e8f459284 and 8bc0be58e, this patch updates call sites of KLOG_EVERY_N_SECS w.r.t. using THROTTLE_MSG to report on the number of suppressed messages. The rationale behind this update is to provide extra information that is useful for troubleshooting, according to my findings while troubleshooting a few recent issues. In addition, I added one micro-test for KLOG_EVERY_N_SECS when used without THROTTLE_MSG and removed useless logging in another test: it seems the logging was added there just for debugging while implementing the test. I also took the liberty of updating corresponding messages and the logging frequency to be more appropriate. Otherwise, this patch doesn't contain any functional modifications. Reviewed-on: http://gerrit.cloudera.org:8080/20855 Tested-by: Kudu Jenkins Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 1214a38e55231ff3aad342ef9efbc326f7a6e659) This is the commit message #11: [docs] updated info on the C++ standard used in Kudu code The Apache Kudu project has switched to using the C++-17 standard a long time ago, so it's time to update the corresponding paragraph in the documentation. Reviewed-on: http://gerrit.cloudera.org:8080/20859 Tested-by: Kudu Jenkins Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 89b7a080c023cac337af7e957e663885d86014f4) This is the commit message #12: Add CLion build directories to .gitignore Reviewed-on: http://gerrit.cloudera.org:8080/20873 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit bfc6d485ea2c1aaa8c864e90de56fd9257a64108) This is the commit message #13: Use GTEST_SKIP() to skip tests GTEST_SKIP() is better than manually logging and return, the former is able to print hint messages and the skipped tests in summaries, for example: $ bin/fs_manager-test --gtest_filter=*TestAddRemoveDataDirs* ... [ RUN ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirs/0 ... Skipping test, file block manager not supported [ SKIPPED ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirs/0 (10 ms) ... [==========] 12 tests from 1 test suite ran. (6407 ms total) [ PASSED ] 6 tests. [ SKIPPED ] 6 tests, listed below: [ SKIPPED ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirs/0 [ SKIPPED ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirs/1 [ SKIPPED ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirs/2 [ SKIPPED ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/0 [ SKIPPED ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/1 [ SKIPPED ] BlockManagerTypes/FsManagerTestBase.TestAddRemoveDataDirsFuzz/2 Reviewed-on: http://gerrit.cloudera.org:8080/20875 Tested-by: Yingchun Lai <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit 264510c877a4ef23b223552af3027a478db51f67) This is the commit message #14: [thirdparty] upgrade boost up to 1.84.0 Upgrade Boost library up to 1.84.0 version. At least, this version properly handles BOOST_NO_CXX98_FUNCTION_BASE macro in boost/config/stdlib/libcpp.hpp, defining it with CLANG 15. Otherwise, it would be necessary to add BOOST_NO_CXX98_FUNCTION_BASE into CFLAGS and CXXFLAGS to avoid compilation errors like the below at least when compiling with CLANG 15: In file included from thirdparty/installed/uninstrumented/include/boost/type_index/stl_type_index.hpp:47: thirdparty/installed/uninstrumented/include/boost/container_hash/hash.hpp:131:33: error: no template named 'unary_function' in namespace 'std'; did you mean '__unary_function'? struct hash_base : std::unary_function<T, std::size_t> {}; ~~~~~^~~~~~~~~~~~~~ __unary_function This patch fixes compilation of the mustache library in the 3rd-party with CLANG 15 which comes with Xcode 15.3 on macOS: std::unary_function has been deprecated since C++11 and removed in C++17 [1], so it was just a matter of time when C++ compilers started to strictly enforce the C++17 standard. [1] https://en.cppreference.com/w/cpp/utility/functional/unary_function Reviewed-on: http://gerrit.cloudera.org:8080/21196 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 56371ef8e4d8e3d7bfa485ab9bec3b222eac285b) This is the commit message #15: [gutil] fix compilation with clang 15 std::{binary,unary}_function have been deprecated since C++11 and removed in C++17 [1], and it seems that CLANG 15 which comes with Xcode 15.3 on macOS enforces this particular part of the C++17 standard. In this instance, it's easy to address the issue because nothing essential was used from std::unary_function and std::binary_function but just typedefs which are no longer needed by the new standard library anyway. Since Kudu switched to C++17 a long time ago [2], the solution is to stop inheriting from the deprecated classes. [1] https://en.cppreference.com/w/cpp/utility/functional/unary_function [2] https://github.com/apache/kudu/commit/e432e637e282ba87d41c02088a801f0838571edc Reviewed-on: http://gerrit.cloudera.org:8080/21206 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 4dc4100fe5cf994cccb5d87cc398e79dde038686) This is the commit message #16: [cmake_modules] recognize linker coming with LLVM 15 This patch updates the GET_LINKER_VERSION cmake's function to work with ld linker that comes with LLVM 15 as a part of Xcode 15.3 on macOS. Without this patch, running cmake in the build directory on macOS using Xcode 15.3 would fail with an error like below: -- Checking linker version when not specifying any flags CMake Error at cmake_modules/KuduLinker.cmake:87 (message): Could not find a suitable linker Reviewed-on: http://gerrit.cloudera.org:8080/21202 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit be36cc0e2edd6a73b7e0936f570612d8c724ac24) This is the commit message #17: [java] Fix build failures Recently it was pointed out that older versions of Kudu are failing when building java due to dependency not present: > Could not resolve all files for configuration ':buildSrc:compileClasspath'. > Could not find gradle-scalafmt-1.14.0.jar (cz.alenkacz:gradle-scalafmt:1.14.0). Searched in the following locations: https://jcenter.bintray.com/cz/alenkacz/gradle-scalafmt/1.14.0/gradle-scalafmt-1.14.0.jar This fix updates the dependency's groupId to be fetched as in Maven Central. The root cause for the issue is jcenter's sun-setting: https://jfrog.com/blog/jcenter-sunset/ https://blog.gradle.org/jcenter-shutdown Reviewed-on: http://gerrit.cloudera.org:8080/21692 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit 42cc1e68eb5bb13e51ba2fde6a1d6483221706f6) This is the commit message #18: [codegen] fix compilation with CLANG 15 on macOS Prior to this fix, compiling Kudu with CLANG 15 on macOS Sonoma would fail with an error like below: [..%] Generating precompiled.ll In file included from src/kudu/codegen/precompiled.cc:39: In file included from src/kudu/common/rowblock.h:21: ... /Applications/Xcode-15.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.4.sdk/usr/include/c++/v1/cstdlib:144:9: error: no member named 'at_quick_exit' in the global namespace using ::at_quick_exit _LIBCPP_USING_IF_EXISTS; ~~^ /Applications/Xcode-15.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.4.sdk/usr/include/c++/v1/cstdlib:145:9: error: no member named 'quick_exit' in the global namespace using ::quick_exit _LIBCPP_USING_IF_EXISTS; Reviewed-on: http://gerrit.cloudera.org:8080/21232 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit 3a856f4502a29f26bb8c9b10a17b06eb16892d02) This is the commit message #19: [style] clean-up on virtual & override specifiers Since Kudu switched to C++17 for non-exported code long time ago, the OVERRIDE macro's usage can be limited to the files comprising the exported API where C++98 compatibility is still required. Also, adding 'virtual' specifier is redundant for virtual methods that override corresponding methods from a base class. One more theme of this patch is updating the code to use explicitly-defaulted constructors and destructors, and adding the 'override' specifier for destructors when appropriate. In addition, many warnings reported by Clang-Tidy were addressed as well. This patch doesn't contain any functional modifications. Reviewed-on: http://gerrit.cloudera.org:8080/20578 Tested-by: Kudu Jenkins Reviewed-by: Yifan Zhang <[email protected]> (cherry picked from commit cd0d93d4dc52a71c3197e6a917761155de32dd78) This is the commit message #20: [fs] fix compilation on macOS After update to the very recent code in from the main branch, compiling the project on my macOS laptop started failing with the following error: Undefined symbols for architecture x86_64: "void google::MakeCheckOpValueString<std::nullptr_t>(std::__1::basic_ostream<char, std::__1::char_traits<char> >*, std::nullptr_t const&)", referenced from: std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >* google::MakeCheckOpString<kudu::fs::LBMCorruptor::Container const*, std::nullptr_t>(kudu::fs::LBMCorruptor::Container const* const&, std::nullptr_t const&, char const*) in log_block_manager-test-util.cc.o ld: symbol(s) not found for architecture x86_64 This patch addresses the issue. I haven't dug any further w.r.t. the difference between compiling on macOS with CLANG and compiling the same on Linux with GCC, but I guess that's something related to the difference in function template instantiation. The information on the C++ compiler is below: $ c++ --version Apple clang version 13.0.0 (clang-1300.0.29.30) Target: x86_64-apple-darwin20.6.0 Thread model: posix InstalledDir: /Applications/Xcode.13.2.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin This is a follow-up to 8c5d195c340360dd962207eaa6969604c65ba9c1. Reviewed-on: http://gerrit.cloudera.org:8080/20618 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit aeaec84df536cbd9a55e5e09998d64a961f5d706) Change-Id: I8167c6824fd56528572fdec5fa2eb2c28c449be0 --- M .gitignore M build-support/ubsan-blacklist.txt M cmake_modules/KuduLinker.cmake M docs/contributing.adoc M java/buildSrc/build.gradle M java/kudu-client/src/main/java/org/apache/kudu/client/KuduRpc.java M src/kudu/benchmarks/tpch/rpc_line_item_dao-test.cc M src/kudu/benchmarks/tpch/rpc_line_item_dao.cc M src/kudu/cfile/binary_dict_block.h M src/kudu/cfile/binary_plain_block.h M src/kudu/cfile/binary_prefix_block.h M src/kudu/cfile/block_encodings.h M src/kudu/cfile/bshuf_block.h M src/kudu/cfile/cfile-test-base.h M src/kudu/cfile/cfile-test.cc M src/kudu/cfile/cfile_reader.h M src/kudu/cfile/encoding-test.cc M src/kudu/cfile/plain_bitmap_block.h M src/kudu/cfile/plain_block.h M src/kudu/cfile/rle_block.h M src/kudu/client/batcher.cc M src/kudu/client/callbacks.h M src/kudu/client/client-test.cc M src/kudu/client/master_rpc.cc M src/kudu/client/master_rpc.h M src/kudu/client/stubs.h M src/kudu/client/write_op.h M src/kudu/codegen/CMakeLists.txt M src/kudu/codegen/code_cache.cc M src/kudu/codegen/row_projector.h M src/kudu/common/generic_iterators.cc M src/kudu/common/iterator.h M src/kudu/consensus/consensus_meta-test.cc M src/kudu/consensus/consensus_peers-test.cc M src/kudu/consensus/consensus_peers.h M src/kudu/consensus/consensus_queue-test.cc M src/kudu/consensus/consensus_queue.h M src/kudu/consensus/log.h M src/kudu/consensus/log_cache-test.cc M src/kudu/consensus/log_index-test.cc M src/kudu/consensus/mt-log-test.cc M src/kudu/consensus/quorum_util-test.cc M src/kudu/consensus/raft_consensus.h M src/kudu/fs/block_manager-stress-test.cc M src/kudu/fs/block_manager-test.cc M src/kudu/fs/block_manager.h M src/kudu/fs/data_dirs-test.cc M src/kudu/fs/fs-test-util.h M src/kudu/fs/fs_manager-test.cc M src/kudu/fs/log_block_manager-test-util.cc M src/kudu/gutil/port.h M src/kudu/gutil/ref_counted_memory.h M src/kudu/gutil/stl_util.h M src/kudu/gutil/strings/human_readable.h M src/kudu/gutil/strings/numbers.h M src/kudu/gutil/strings/util.h M src/kudu/gutil/threading/thread_collision_warner.h M src/kudu/integration-tests/all_types-itest.cc M src/kudu/integration-tests/alter_table-test.cc M src/kudu/integration-tests/consistency-itest.cc M src/kudu/integration-tests/create-table-stress-test.cc M src/kudu/integration-tests/disk_failure-itest.cc M src/kudu/integration-tests/full_stack-insert-scan-test.cc M src/kudu/integration-tests/linked_list-test.cc M src/kudu/integration-tests/location_assignment-itest.cc M src/kudu/integration-tests/master_authz-itest.cc M src/kudu/integration-tests/master_failover-itest.cc M src/kudu/integration-tests/master_replication-itest.cc M src/kudu/integration-tests/tablet_history_gc-itest.cc M src/kudu/integration-tests/ts_authz-itest.cc M src/kudu/integration-tests/ts_recovery-itest.cc M src/kudu/integration-tests/update_scan_delta_compact-test.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master_service.cc M src/kudu/master/ts_manager.cc M src/kudu/mini-cluster/external_mini_cluster.h M src/kudu/mini-cluster/internal_mini_cluster.h M src/kudu/rpc/acceptor_pool.cc M src/kudu/rpc/acceptor_pool.h M src/kudu/rpc/connection.cc M src/kudu/rpc/exactly_once_rpc-test.cc M src/kudu/rpc/messenger.cc M src/kudu/rpc/messenger.h M src/kudu/rpc/mt-rpc-test.cc M src/kudu/rpc/periodic-test.cc M src/kudu/rpc/protoc-gen-krpc.cc M src/kudu/rpc/proxy.cc M src/kudu/rpc/retriable_rpc.h M src/kudu/rpc/rpc-test.cc M src/kudu/rpc/rpc.h M src/kudu/rpc/rpc_service.h M src/kudu/rpc/rpc_sidecar.h M src/kudu/rpc/server_negotiation.cc M src/kudu/rpc/service_if.h M src/kudu/rpc/service_pool.h M src/kudu/rpc/transfer.h M src/kudu/security/token-test.cc M src/kudu/security/token_verifier.cc M src/kudu/server/generic_service.h M src/kudu/server/glog_metrics.cc M src/kudu/server/monitored_task.h M src/kudu/server/server_base.cc M src/kudu/server/webserver-test.cc M src/kudu/server/webserver.cc M src/kudu/tablet/cfile_set-test.cc M src/kudu/tablet/cfile_set.h M src/kudu/tablet/compaction-test.cc M src/kudu/tablet/compaction.cc M src/kudu/tablet/compaction.h M src/kudu/tablet/composite-pushdown-test.cc M src/kudu/tablet/delta_compaction-test.cc M src/kudu/tablet/delta_store.h M src/kudu/tablet/deltafile-test.cc M src/kudu/tablet/deltamemstore.h M src/kudu/tablet/diskrowset.h M src/kudu/tablet/major_delta_compaction-test.cc M src/kudu/tablet/memrowset.cc M src/kudu/tablet/memrowset.h M src/kudu/tablet/ops/op.h M src/kudu/tablet/ops/op_driver.cc M src/kudu/tablet/rowset.h M src/kudu/tablet/rowset_info.cc M src/kudu/tablet/tablet-decoder-eval-test.cc M src/kudu/tablet/tablet-test-util.h M src/kudu/tablet/tablet-test.cc M src/kudu/tablet/tablet.h M src/kudu/tablet/tablet_bootstrap-test.cc M src/kudu/tablet/tablet_history_gc-test.cc M src/kudu/tablet/tablet_metadata-test.cc M src/kudu/tablet/tablet_mm_ops-test.cc M src/kudu/tablet/tablet_replica-test.cc M src/kudu/tablet/tablet_replica.cc M src/kudu/tablet/tablet_replica.h M src/kudu/tablet/tablet_throttle-test.cc M src/kudu/tablet/txn_coordinator.h M src/kudu/thrift/sasl_client_transport.cc M src/kudu/tools/create-table-tool-test.cc M src/kudu/tools/diagnostics_log_parser.h M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck.h M src/kudu/tools/ksck_remote-test.cc M src/kudu/tools/ksck_remote.h M src/kudu/tools/kudu-tool-test.cc M src/kudu/tserver/tablet_copy_service-test.cc M src/kudu/tserver/tablet_replica_lookup.h M src/kudu/tserver/tablet_server-stress-test.cc M src/kudu/tserver/tablet_server-test.cc M src/kudu/tserver/tablet_service.cc M src/kudu/tserver/tablet_service.h M src/kudu/tserver/ts_tablet_manager-test.cc M src/kudu/tserver/ts_tablet_manager.h M src/kudu/util/async_util-test.cc M src/kudu/util/bit-stream-utils.inline.h M src/kudu/util/cache.cc M src/kudu/util/cache.h M src/kudu/util/cache_metrics.h M src/kudu/util/cloud/instance_metadata.h M src/kudu/util/compression/compression_codec.cc M src/kudu/util/compression/compression_codec.h M src/kudu/util/debug/trace_event.h M src/kudu/util/debug/trace_event_impl.cc M src/kudu/util/debug/trace_event_impl.h M src/kudu/util/debug/trace_event_synthetic_delay.cc M src/kudu/util/env-test.cc M src/kudu/util/env.h M src/kudu/util/env_posix.cc M src/kudu/util/flags.cc M src/kudu/util/hdr_histogram.h M src/kudu/util/interval_tree-test.cc M src/kudu/util/jsonwriter.cc M src/kudu/util/jwt.h M src/kudu/util/logging-test.cc M src/kudu/util/logging.cc M src/kudu/util/maintenance_manager.cc M src/kudu/util/memory/memory.h M src/kudu/util/metrics.cc M src/kudu/util/metrics.h M src/kudu/util/minidump-test.cc M src/kudu/util/nvm_cache.cc M src/kudu/util/pb_util-internal.h M src/kudu/util/pb_util-test.cc M src/kudu/util/protoc-gen-insertions.cc M src/kudu/util/rolling_log-test.cc M src/kudu/util/stack_watchdog-test.cc M src/kudu/util/test_util.h M src/kudu/util/threadpool-test.cc M src/kudu/util/trace-test.cc M src/kudu/util/web_callback_registry.h M thirdparty/download-thirdparty.sh A thirdparty/patches/boost-bootstrap.patch M thirdparty/vars.sh 192 files changed, 1,732 insertions(+), 1,226 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/21769/1 -- To view, visit http://gerrit.cloudera.org:8080/21769 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: newchange Gerrit-Change-Id: I8167c6824fd56528572fdec5fa2eb2c28c449be0 Gerrit-Change-Number: 21769 Gerrit-PatchSet: 1 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]>
