[CARBONDATA-4196] Allow zero or more white space in GEO UDFs #4145
Conversation
nihal0107
changed the title
|
Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_2.4.5/3753/ |
|
Build Failed with Spark 2.3.4, Please check CI http://121.244.95.60:12602/job/ApacheCarbonPRBuilder2.3/5496/ |
|
retest this please. |
integration/spark/src/test/scala/org/apache/carbondata/geo/GeoTest.scala
Outdated
Show resolved
Hide resolved
|
Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_2.4.5/3755/ |
|
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12602/job/ApacheCarbonPRBuilder2.3/5498/ |
|
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12602/job/ApacheCarbonPRBuilder2.3/5501/ |
|
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_2.4.5/3758/ |
|
LGTM |
|
retest this please |
|
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_2.4.5/3767/ |
|
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12602/job/ApacheCarbonPRBuilder2.3/5510/ |
Why is this PR needed?
Currently, regex of geo UDF is not allowing zero space between UDF name and parenthesis. It always expects a single space in between. For ex:
linestring (120.184179 30.327465). Because of this sometimes using the UDFs without space is not giving the expected result.What changes were proposed in this PR?
Allow zero space between UDFs and parenthesis.
Does this PR introduce any user interface change?
Is any new testcase added?