crunch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nathan Schile (JIRA)" <>
Subject [jira] [Created] (CRUNCH-623) Improve PTable#cogroup Javadoc
Date Thu, 29 Sep 2016 21:23:20 GMT
Nathan Schile created CRUNCH-623:

             Summary: Improve PTable#cogroup Javadoc
                 Key: CRUNCH-623
             Project: Crunch
          Issue Type: Improvement
          Components: Core
            Reporter: Nathan Schile
            Assignee: Josh Wills
            Priority: Trivial

The Javadoc for PTable#cogroup maybe confusing when comparing it to the PTable#join javadoc

   * Perform an inner join on this table and the one passed in as an argument on
   * their common keys.
  <U> PTable<K, Pair<V, U>> join(PTable<K, U> other);

   * Co-group operation with the given table on common keys.
  <U> PTable<K, Pair<Collection<V>, Collection<U>>> cogroup(PTable<K,
U> other);

The phrasing of "common keys" in the cogroup method make it sound like the cogroup will be
a inner join on the table. I think removing the "common keys" from the cogroup method will
remove that confusion.

This message was sent by Atlassian JIRA

View raw message