fjtirado commented on code in PR #2161:
URL: 
https://github.com/apache/incubator-kie-kogito-apps/pull/2161#discussion_r1884008217


##########
data-index/data-index-common/src/main/java/org/kie/kogito/index/json/JsonUtils.java:
##########
@@ -58,4 +59,11 @@ private static ObjectNode createObjectNode(String 
variableName, Object variableV
         }
         return result;
     }
+
+    public static <T> AttributeFilter<T> jsonFilter(AttributeFilter<T> filter) 
{

Review Comment:
   This might be consider a bit hacky, but the alternative to not include 
graphql type info will be, in the case of postgresql implementation, to either 
hardcode the column or search in the metamodel, we can avoid that by just 
including this flag, which might be useful for other implementations that are 
not postgres too. 



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to