[GitHub] [carbondata] ravipesala commented on a change in pull request #3329: [CARBONDATA-3474]Fix validate mvQuery having filter expression and correct error message

[GitBox]

ravipesala commented on a change in pull request #3329: [CARBONDATA-3474]Fix validate mvQuery having filter expression and correct error message
URL: https://github.com/apache/carbondata/pull/3329#discussion_r304754419
 
 

 ##########
 File path: datamap/mv/core/src/main/scala/org/apache/carbondata/mv/datamap/MVUtil.scala
 ##########
 @@ -314,7 +314,8 @@ class MVUtil {
     if (a.child.isInstanceOf[GetMapValue] || a.child.isInstanceOf[GetStructField] ||
         a.child.isInstanceOf[GetArrayItem]) {
       throw new UnsupportedOperationException(
-        s"MV datamap is unsupported for ComplexData type child column: " + a.child.simpleString)
+        s"MV datamap is unsupported for ComplexData type child column / if return type is of " +
+        s"complex type for function: " + a.child.simpleString)
 
 Review comment:
   Better format as below.
   ```
   MV datamap is not supported for complex datatype columns  and complex datatype return types of function :
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services