----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9683/#review17238 -----------------------------------------------------------
client/src/main/java/org/apache/sqoop/client/request/Request.java <https://reviews.apache.org/r/9683/#comment36626> Move this constant value to common's SqoopProtocolConstants.java. server/src/main/java/org/apache/sqoop/server/RequestContext.java <https://reviews.apache.org/r/9683/#comment36627> Pass constants - vasanthkumar On March 1, 2013, 8:58 a.m., Abraham Elmahrek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9683/ > ----------------------------------------------------------- > > (Updated March 1, 2013, 8:58 a.m.) > > > Review request for Sqoop and Jarek Cecho. > > > Description > ------- > > commit cbbd044f7b3da77741f50c5a5082b57a3b49c1e2 > Author: Abraham Elmahrek <[email protected]> > Date: Fri Mar 1 00:56:18 2013 -0800 > > SQOOP-664 Send user name as HTTP header in client to server communication > > Using a custom HTTP header "User-Name" following convention defined in > rfc-6648. > User name of user executing sqoop is sent to the server. > > :100644 100644 1720507... 1b5d34c... M > client/src/main/java/org/apache/sqoop/client/request/Request.java > :100644 100644 c6b6569... 9685d96... M > server/src/main/java/org/apache/sqoop/server/RequestContext.java > > > This addresses bug SQOOP-664. > https://issues.apache.org/jira/browse/SQOOP-664 > > > Diffs > ----- > > client/src/main/java/org/apache/sqoop/client/request/Request.java > 17205070b87ac1904c055ba2d4fa7b773f2f3034 > server/src/main/java/org/apache/sqoop/server/RequestContext.java > c6b6569a9c19860fe8d1138040c1ffaa1ce5844e > > Diff: https://reviews.apache.org/r/9683/diff/ > > > Testing > ------- > > Ran all unit tests and verified client is sending header to server. > > > Thanks, > > Abraham Elmahrek > >
