You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Reynold Xin (JIRA)" <ji...@apache.org> on 2015/07/08 22:04:04 UTC

[jira] [Created] (SPARK-8907) Speed up path construction in DynamicPartitionWriterContainer.outputWriterForRow

Reynold Xin created SPARK-8907:
----------------------------------

             Summary: Speed up path construction in DynamicPartitionWriterContainer.outputWriterForRow
                 Key: SPARK-8907
                 URL: https://issues.apache.org/jira/browse/SPARK-8907
             Project: Spark
          Issue Type: Sub-task
          Components: SQL
            Reporter: Reynold Xin


Don't use zip and scala collection methods to avoid garbage collection

{code}
    val partitionPath = partitionColumns.zip(row.toSeq).map { case (col, rawValue) =>
      val string = if (rawValue == null) null else String.valueOf(rawValue)
      val valueString = if (string == null || string.isEmpty) {
        defaultPartitionName
      } else {
        PartitioningUtils.escapePathName(string)
      }
      s"/$col=$valueString"
    }.mkString.stripPrefix(Path.SEPARATOR)
{code}

We can probably use catalyst expressions themselves to construct the path, and then we can leverage code generation to do this.






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

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org