----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68710/#review208621 -----------------------------------------------------------
Ship it! LGTM +! (non-binding) - Laszlo Pinter On Sept. 13, 2018, 10:11 a.m., Karen Coppage wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68710/ > ----------------------------------------------------------- > > (Updated Sept. 13, 2018, 10:11 a.m.) > > > Review request for hive and Laszlo Pinter. > > > Bugs: HIVE-20544 > https://issues.apache.org/jira/browse/HIVE-20544 > > > Repository: hive-git > > > Description > ------- > > TOpenSessionReq, if client protocol is unset, both username and password are > logged. Logging a password is a security risk. This patch would hide it with > asterisks. > > > Diffs > ----- > > > service-rpc/src/gen/thrift/gen-javabean/org/apache/hive/service/rpc/thrift/TOpenSessionReq.java > 3195e704f3 > > > Diff: https://reviews.apache.org/r/68710/diff/2/ > > > Testing > ------- > > > Thanks, > > Karen Coppage > >
