[ 
https://issues.apache.org/jira/browse/CASSANDRA-910?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12848307#action_12848307
 ] 

Jonathan Ellis commented on CASSANDRA-910:
------------------------------------------

+    public Configuration getConf()
+    {
+        return conf;
+    }
+
+    public void setConf(Configuration conf)
+    {
+        this.conf = conf;
+    }

I am not a hadoop expert, but this doesn't feel right to me.  If you were 
supposed to use one IF per Configuration why pass a JobContext to getSplits?  
(I don't see any actual uses of getConf() in the patch, either.)  If pig needs 
this somewhere, how about adding getConfiguration() to CassandraStorage class?  
(Anyone who wants the configuration would need a handle to a CS reference 
anyway, to call getInputFormat.)

> Pig LoadFunc for Cassandra
> --------------------------
>
>                 Key: CASSANDRA-910
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-910
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Contrib
>    Affects Versions: 0.6
>            Reporter: Stu Hood
>             Fix For: 0.6
>
>         Attachments: 0001-Pig-loadfunc-for-Pig-0.7.0-dev.patch, 
> 0001-Pig-loadfunc-for-Pig-0.7.0-dev.patch, pig-0.7.0-dev-r923931.jar
>
>
> Pig provides a clean relational language for offline processing of large 
> datasets, and since we already have Hadoop MapReduce support, adding support 
> for Pig is trivial.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to