jcamachor commented on a change in pull request #829: Hive JDBC Storage 
Handler: Support For Writing Data to JDBC Data Source
URL: https://github.com/apache/hive/pull/829#discussion_r339794521
 
 

 ##########
 File path: 
jdbc-handler/src/main/java/org/apache/hive/storage/jdbc/JdbcOutputFormat.java
 ##########
 @@ -18,19 +18,25 @@
 import org.apache.hadoop.fs.Path;
 import org.apache.hadoop.hive.ql.exec.FileSinkOperator.RecordWriter;
 import org.apache.hadoop.hive.ql.io.HiveOutputFormat;
+import org.apache.hadoop.hive.shims.ShimLoader;
 import org.apache.hadoop.io.MapWritable;
 import org.apache.hadoop.io.NullWritable;
 import org.apache.hadoop.io.Writable;
 import org.apache.hadoop.mapred.JobConf;
 import org.apache.hadoop.mapred.OutputFormat;
+import org.apache.hadoop.mapreduce.TaskAttemptContext;
 import org.apache.hadoop.util.Progressable;
+import org.apache.hive.storage.jdbc.dao.GenericJdbcDatabaseAccessor;
 
 import java.io.IOException;
 import java.util.Properties;
 
 public class JdbcOutputFormat implements OutputFormat<NullWritable, 
MapWritable>,
                                          HiveOutputFormat<NullWritable, 
MapWritable> {
 
+  private org.apache.hadoop.mapreduce.RecordWriter recordWriter;
 
 Review comment:
   These two objects can be moved to `public RecordWriter getHiveRecordWriter`, 
since they are not being accessed outside of the scope of that method.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to