cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Ellis (JIRA)" <j...@apache.org>
Subject [jira] Commented: (CASSANDRA-910) Pig LoadFunc for Cassandra
Date Mon, 22 Mar 2010 20:03:27 GMT

    [ 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.


Mime
View raw message