Repository: incubator-s2graph
Updated Branches:
  refs/heads/master ab032836f -> 6082be918


[S2GRAPH-42]: Bug on EdgeTransformer with specific cases.

  add pad to deal with different size on splitted tokens by on place holder and 
passed values on transform.
  ex) [["age_band.slot.$.8", "age_band"]] should give 
"age_band.slot.${age_band}.8", not losing last ".8".

JIRA:
  [S2GRAPH-42] https://issues.apache.org/jira/browse/S2GRAPH-42

Pull Request:
  Closes #25


Project: http://git-wip-us.apache.org/repos/asf/incubator-s2graph/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-s2graph/commit/6082be91
Tree: http://git-wip-us.apache.org/repos/asf/incubator-s2graph/tree/6082be91
Diff: http://git-wip-us.apache.org/repos/asf/incubator-s2graph/diff/6082be91

Branch: refs/heads/master
Commit: 6082be91874219a1af1963c688cc382938a00545
Parents: ab03283
Author: DO YUNG YOON <[email protected]>
Authored: Tue Feb 23 17:29:27 2016 +0900
Committer: DO YUNG YOON <[email protected]>
Committed: Tue Feb 23 17:29:27 2016 +0900

----------------------------------------------------------------------
 CHANGES                                                       | 2 ++
 s2core/src/main/scala/com/kakao/s2graph/core/QueryParam.scala | 3 ++-
 2 files changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-s2graph/blob/6082be91/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index d4cb091..8278b26 100644
--- a/CHANGES
+++ b/CHANGES
@@ -42,6 +42,8 @@ Release 0.12.1 - unreleased
     
     S2GRAPH-38: Transform option gives wrong result since it transform degree 
edge too (Committed by DOYUNG YOON).
 
+    S2GRAPH-42: Bug on EdgeTransformer with specific cases (Committed by 
DOYUNG YOON).
+
   TASKS
 
     S2GRAPH-2: Update document and quick start environment to have recent 
set-up command changes.

http://git-wip-us.apache.org/repos/asf/incubator-s2graph/blob/6082be91/s2core/src/main/scala/com/kakao/s2graph/core/QueryParam.scala
----------------------------------------------------------------------
diff --git a/s2core/src/main/scala/com/kakao/s2graph/core/QueryParam.scala 
b/s2core/src/main/scala/com/kakao/s2graph/core/QueryParam.scala
index b0f564f..3f0dfbe 100644
--- a/s2core/src/main/scala/com/kakao/s2graph/core/QueryParam.scala
+++ b/s2core/src/main/scala/com/kakao/s2graph/core/QueryParam.scala
@@ -120,7 +120,8 @@ case class EdgeTransformer(queryParam: QueryParam, jsValue: 
JsValue) {
               nextStepOpt: Option[Step]): Seq[InnerValLike] = {
 
     val tokens = fmt.split(Delimiter)
-    val mergedStr = tokens.zip(values).map { case (prefix, innerVal) => prefix 
+ innerVal.toString }.mkString
+    val _values = values.padTo(tokens.length, InnerVal.withStr("", 
queryParam.label.schemaVersion))
+    val mergedStr = tokens.zip(_values).map { case (prefix, innerVal) => 
prefix + innerVal.toString }.mkString
     //    logger.error(s"${tokens.toList}, ${values}, $mergedStr")
     //    println(s"${tokens.toList}, ${values}, $mergedStr")
     nextStepOpt match {

Reply via email to