[jira] [Commented] (SPARK-17169) To use scala macros to update code when SharedParamsCodeGen.scala changed
[ https://issues.apache.org/jira/browse/SPARK-17169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16196206#comment-16196206 ] Hyukjin Kwon commented on SPARK-17169: -- [~josephkb], should we maybe leave this as {{Won't Fix}} or {{Later}} if you don't think it is worthwhile and there is no explicit objection against this (it looks so)? > To use scala macros to update code when SharedParamsCodeGen.scala changed > - > > Key: SPARK-17169 > URL: https://issues.apache.org/jira/browse/SPARK-17169 > Project: Spark > Issue Type: Improvement > Components: ML >Reporter: Qian Huang >Priority: Minor > > As commented in the file SharedParamsCodeGen.scala, we have to manually run > build/sbt "mllib/runMain org.apache.spark.ml.param.shared.SharedParamsCodeGen" > to generate and update it. > It could be better to do compile time code-gen for this using scala macros > rather than running the script as described above. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Commented] (SPARK-17169) To use scala macros to update code when SharedParamsCodeGen.scala changed
[ https://issues.apache.org/jira/browse/SPARK-17169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15798989#comment-15798989 ] Joseph K. Bradley commented on SPARK-17169: --- Is this worthwhile? A lot of developers aren't familiar with Scala macros, and macros can get pretty obscure if we're not careful. > To use scala macros to update code when SharedParamsCodeGen.scala changed > - > > Key: SPARK-17169 > URL: https://issues.apache.org/jira/browse/SPARK-17169 > Project: Spark > Issue Type: Improvement > Components: ML >Reporter: Qian Huang >Priority: Minor > > As commented in the file SharedParamsCodeGen.scala, we have to manually run > build/sbt "mllib/runMain org.apache.spark.ml.param.shared.SharedParamsCodeGen" > to generate and update it. > It could be better to do compile time code-gen for this using scala macros > rather than running the script as described above. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Commented] (SPARK-17169) To use scala macros to update code when SharedParamsCodeGen.scala changed
[ https://issues.apache.org/jira/browse/SPARK-17169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15436402#comment-15436402 ] Weichen Xu commented on SPARK-17169: I will work on it and create pr soon! > To use scala macros to update code when SharedParamsCodeGen.scala changed > - > > Key: SPARK-17169 > URL: https://issues.apache.org/jira/browse/SPARK-17169 > Project: Spark > Issue Type: Improvement > Components: ML >Reporter: Qian Huang >Priority: Minor > > As commented in the file SharedParamsCodeGen.scala, we have to manually run > build/sbt "mllib/runMain org.apache.spark.ml.param.shared.SharedParamsCodeGen" > to generate and update it. > It could be better to do compile time code-gen for this using scala macros > rather than running the script as described above. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org
[jira] [Commented] (SPARK-17169) To use scala macros to update code when SharedParamsCodeGen.scala changed
[ https://issues.apache.org/jira/browse/SPARK-17169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15430282#comment-15430282 ] Yanbo Liang commented on SPARK-17169: - Meanwhile, it's better we can do compile time code-gen for python params as well, that is to say run {{python _shared_params_code_gen.py > shared.py}} automatically. > To use scala macros to update code when SharedParamsCodeGen.scala changed > - > > Key: SPARK-17169 > URL: https://issues.apache.org/jira/browse/SPARK-17169 > Project: Spark > Issue Type: Improvement > Components: ML >Reporter: Qian Huang >Priority: Minor > > As commented in the file SharedParamsCodeGen.scala, we have to manually run > build/sbt "mllib/runMain org.apache.spark.ml.param.shared.SharedParamsCodeGen" > to generate and update it. > It could be better to do compile time code-gen for this using scala macros > rather than running the script as described above. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org For additional commands, e-mail: issues-h...@spark.apache.org