[ 
https://issues.apache.org/jira/browse/MAHOUT-1660?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14582473#comment-14582473
 ] 

ASF GitHub Bot commented on MAHOUT-1660:
----------------------------------------

Github user andrewmusselman commented on a diff in the pull request:

    https://github.com/apache/mahout/pull/135#discussion_r32264003
  
    --- Diff: 
math-scala/src/main/scala/org/apache/mahout/math/drm/package.scala ---
    @@ -115,6 +121,46 @@ package object drm {
         }
       }
     
    +  /**
    +   * Convert arbitrarily-keyed matrix to int-keyed matrix. Some algebra 
will accept only int-numbered
    +   * row matrices. So this method is to help.
    +   *
    +   * @param drmX input to be transcoded
    +   * @param computeMap collect `old key -> int key` map to front-end?
    +   * @tparam K key type
    +   * @return Sequentially keyed matrix + (optionally) map from non-int key 
to [[Int]] key. If the
    +   *         key type is actually Int, then we just return the argument 
with None for the map,
    +   *         regardless of computeMap parameter.
    +   */
    +  def drm2IntKeyed[K: ClassTag](drmX: DrmLike[K], computeMap: Boolean = 
false): (DrmLike[Int], Option[DrmLike[K]]) =
    --- End diff --
    
    This looks like a pass-through to DistributedEngine; am I reading it right 
or missing something?


> Hadoop1HDFSUtil.readDRMHEader should be taking Hadoop conf
> ----------------------------------------------------------
>
>                 Key: MAHOUT-1660
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-1660
>             Project: Mahout
>          Issue Type: Bug
>          Components: spark
>    Affects Versions: 0.10.0
>            Reporter: Suneel Marthi
>            Assignee: Dmitriy Lyubimov
>            Priority: Minor
>             Fix For: 0.10.2
>
>
> Hadoop1HDFSUtil.readDRMHEader should be taking Hadoop configuration from 
> Context and not ignore it



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to