Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/19873 )

Change subject: KUDU-3448 Move password retrieval to master init
......................................................................

KUDU-3448 Move password retrieval to master init

Apparently, forks can be expensive, at least on some systems, which
makes fetching the password to encrypt/decrypt key materials whenever a
TSK or IPKI is generated or loaded into memory not ideal as it may cause
the server process to hang.

This patch moves the password retrieval to the master initialization
phase and the passwords are now stored in memory for later use.

Change-Id: I746e657ae8d295f5f34225d63686beea1dff6b7c
Reviewed-on: http://gerrit.cloudera.org:8080/19873
Tested-by: Attila Bukor <abu...@apache.org>
Reviewed-by: Alexey Serbin <ale...@apache.org>
---
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
M src/kudu/master/master.cc
M src/kudu/security/token_signer.cc
M src/kudu/security/token_signer.h
M src/kudu/security/token_signing_key.cc
M src/kudu/security/token_signing_key.h
7 files changed, 82 insertions(+), 41 deletions(-)

Approvals:
  Attila Bukor: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/19873
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I746e657ae8d295f5f34225d63686beea1dff6b7c
Gerrit-Change-Number: 19873
Gerrit-PatchSet: 6
Gerrit-Owner: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <greber...@gmail.com>
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Zoltan Chovan <zcho...@cloudera.com>

Reply via email to