> On Aug. 5, 2014, 11:27 p.m., Abraham Elmahrek wrote: > > src/java/org/apache/sqoop/mapreduce/SQLServerExportDBExecThread.java, line > > 128 > > <https://reviews.apache.org/r/24242/diff/1/?file=650524#file650524line128> > > > > Should this have a semicolon at the end of it? > > > > Also, does it make any sense to separate this out into its own > > statement? > > Keegan Witt wrote: > I'm not sure I follow. If I add a semicolon, this will be two statements > in a single PreparedStatement (which shouldn't be done). I'd be OK breaking > them into separate PreparedStatements, but I didn't see where that would be > done. The AsyncSqlRecordWriter interface just has getPreparedStatement() > which returns a single PreparedStatement, not a collection of > PreparedStatements to run. What am I missing?
Good points. It seems like there is a "configureConnection" method that can be overriden in the parent class "SQLServerAsyncDBExecThread". Would that work? > On Aug. 5, 2014, 11:27 p.m., Abraham Elmahrek wrote: > > src/java/org/apache/sqoop/mapreduce/sqlserver/SqlServerExportBatchOutputFormat.java, > > line 67 > > <https://reviews.apache.org/r/24242/diff/1/?file=650525#file650525line67> > > > > Should this have a semicolon at the end of it? > > > > Also, does it make any sense to separate this out into its own > > statement? It seems like this one can be placed in the "SqlServerExportBatchRecordWriter" constructor. Would that work? - Abraham ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24242/#review49657 ----------------------------------------------------------- On Aug. 4, 2014, 5:48 p.m., Keegan Witt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24242/ > ----------------------------------------------------------- > > (Updated Aug. 4, 2014, 5:48 p.m.) > > > Review request for Sqoop. > > > Repository: sqoop-trunk > > > Description > ------- > > Adds an extra arg for SQL Server that runs SET IDENTITY_INSERT <table> ON > before exporting the data. > > > Diffs > ----- > > src/docs/user/connectors.txt bba946a > src/java/org/apache/sqoop/manager/SQLServerManager.java 534c0cc > src/java/org/apache/sqoop/mapreduce/SQLServerExportDBExecThread.java > 1810ba8 > > src/java/org/apache/sqoop/mapreduce/sqlserver/SqlServerExportBatchOutputFormat.java > cc69d77 > > Diff: https://reviews.apache.org/r/24242/diff/ > > > Testing > ------- > > Tested that change enables inserts on tables with identity (with SQL Server > 2005). > > > Thanks, > > Keegan Witt > >
