----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4705/#review6883 -----------------------------------------------------------
src/java/org/apache/hcatalog/mapreduce/Security.java <https://reviews.apache.org/r/4705/#comment15329> It would be good to have a WeakReference for the Token, so that the tokenMap does not hold tokens of Jobs out of reference. src/java/org/apache/hcatalog/mapreduce/Security.java <https://reviews.apache.org/r/4705/#comment15331> Applying the patch on branch 0.4 fails with [javac] symbol : method buildTokenService(java.net.InetSocketAddress) [javac] location: class org.apache.hadoop.security.SecurityUtil [javac] jtTokenSelector.selectToken(org.apache.hadoop.security.SecurityUtil.buildTokenService(JobTracker.getAddress(conf)), - Rohini On 2012-04-12 01:43:58, Francis Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4705/ > ----------------------------------------------------------- > > (Updated 2012-04-12 01:43:58) > > > Review request for hcatalog, Ashutosh Chauhan and Sushanth Sowmyan. > > > Summary > ------- > > Cleaned up a bit and move some redundant code in the Security.java to have > better encapsulation of security stuff. Removed jt token cancellation as this > will be taken care of by jt. > > > This addresses bug HCATALOG-69. > https://issues.apache.org/jira/browse/HCATALOG-69 > > > Diffs > ----- > > src/java/org/apache/hcatalog/common/HCatConstants.java 264ecbd > src/java/org/apache/hcatalog/common/HCatUtil.java 46af296 > src/java/org/apache/hcatalog/mapreduce/FileOutputCommitterContainer.java > 69e12f3 > src/java/org/apache/hcatalog/mapreduce/HCatBaseOutputFormat.java 5bb46d1 > src/java/org/apache/hcatalog/mapreduce/HCatOutputFormat.java a48332a > src/java/org/apache/hcatalog/mapreduce/Security.java db998a1 > src/java/org/apache/hcatalog/pig/HCatStorer.java 34d8ec3 > > Diff: https://reviews.apache.org/r/4705/diff > > > Testing > ------- > > e2e tests pass (except for hbase related ones, need to fix my harness) > > > Thanks, > > Francis > >
