> On March 28, 2019, 12:37 p.m., Benjamin Bannier wrote: > > src/tests/csi_client_tests.cpp > > Line 86 (original), 83 (patched) > > <https://reviews.apache.org/r/70321/diff/2/?file=2135963#file2135963line90> > > > > Not started here, but I believe we strive to distinguish members by > > appended underscores, not locals.
Quoted from our C++ style guide: * We prepend constructor and function arguments with a leading underscore to avoid ambiguity and / or shadowing. * Prefer trailing underscores for use as member fields (but not required). Some trailing underscores are used to distinguish between similar variables in the same scope (think prime symbols), but this should be avoided as much as possible, including removing existing instances in the code base. My interpretation is that: 1. `_connection` is used in arguments. 2. `connection_` can be used in member variables. 3. `connection_` can be used to distinguish variables in the same scope, but should be avoided. That said, since this is not related to this patch, let me just revert this change. - Chun-Hung ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70321/#review214173 ----------------------------------------------------------- On March 28, 2019, 8:02 a.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70321/ > ----------------------------------------------------------- > > (Updated March 28, 2019, 8:02 a.m.) > > > Review request for mesos, Benjamin Bannier, Jie Yu, and Jan Schlicht. > > > Bugs: MESOS-9624 > https://issues.apache.org/jira/browse/MESOS-9624 > > > Repository: mesos > > > Description > ------- > > Since per-CSI-call metrics are no longer implemented, there is very less > value to keep the `mesos::csi::v0::RPC` enum, which is tightly coupled > with `mesos::csi::v0::Client`. Therefore, the enum and its header are > removed. > > The header and implementation file for `mesos::csi::v0::Client` is also > renamed for future CSI v1 support. > > > Diffs > ----- > > src/CMakeLists.txt 3397c3b1d4e8a7900b2e5f870679cc7aa30b4be2 > src/Makefile.am bcafe48b2105575371464a29783bc6f3f1c2cf8d > src/csi/client.hpp c2583cf4c0d2abc38d65ddc801cae3696a8080a9 > src/csi/client.cpp 9e17f5bcb902470045dac704f2471d2962336e48 > src/csi/rpc.hpp b2502ceb319638038b4d151965f6226db675f96b > src/csi/rpc.cpp e8a2770a6d8ae48a717ed93fa872e9edee45e618 > src/csi/service_manager.cpp PRE-CREATION > src/csi/v0_client.hpp PRE-CREATION > src/csi/v0_volume_manager.cpp PRE-CREATION > src/csi/v0_volume_manager_process.hpp PRE-CREATION > src/tests/csi_client_tests.cpp c8f3f04e6bff40d62dd0f85fcc146d71fa9e5d34 > src/tests/storage_local_resource_provider_tests.cpp > 797f89e3545965e2cf2fd5ec0ecd78fc77a4ea87 > > > Diff: https://reviews.apache.org/r/70321/diff/2/ > > > Testing > ------- > > make check > > > Thanks, > > Chun-Hung Hsiao > >
