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

    https://github.com/apache/carbondata/pull/1618#discussion_r155165332
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonEnv.scala ---
    @@ -54,6 +54,7 @@ class CarbonEnv {
     
       def init(sparkSession: SparkSession): Unit = {
         sparkSession.udf.register("getTupleId", () => "")
    +    sparkSession.udf.register("getPositionId", () => "")
    --- End diff --
    
    it is not a good idea to add here, better add in CarbonEnvPreListner to 
register


---

Reply via email to