----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62360/#review185647 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/parse/repl/CopyUtils.java Line 61 (original), 69 (patched) <https://reviews.apache.org/r/62360/#comment261964> use lambdas ql/src/java/org/apache/hadoop/hive/ql/parse/repl/CopyUtils.java Lines 82 (patched) <https://reviews.apache.org/r/62360/#comment261965> Path not required on right hand side ql/src/java/org/apache/hadoop/hive/ql/parse/repl/CopyUtils.java Lines 153 (patched) <https://reviews.apache.org/r/62360/#comment261966> use lambda - anishek On Sept. 19, 2017, 12:14 a.m., Daniel Dai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62360/ > ----------------------------------------------------------- > > (Updated Sept. 19, 2017, 12:14 a.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > See HIVE-16898 > > > Diffs > ----- > > metastore/src/java/org/apache/hadoop/hive/metastore/ReplChangeManager.java > 88d6a7a > ql/src/java/org/apache/hadoop/hive/ql/exec/ReplCopyTask.java 54746d3 > ql/src/java/org/apache/hadoop/hive/ql/parse/repl/CopyUtils.java 28e7bcb > > > Diff: https://reviews.apache.org/r/62360/diff/3/ > > > Testing > ------- > > Manually test it with debugger: setup a breakpoint right before copy, and > drop table in another session. > > > Thanks, > > Daniel Dai > >
