You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by GitBox <gi...@apache.org> on 2019/07/18 06:36:36 UTC

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

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:
users@infra.apache.org


With regards,
Apache Git Services