[ 
https://issues.apache.org/jira/browse/PHOENIX-7073?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17788205#comment-17788205
 ] 

ASF GitHub Bot commented on PHOENIX-7073:
-----------------------------------------

ranganathg commented on code in PR #1712:
URL: https://github.com/apache/phoenix/pull/1712#discussion_r1399922953


##########
phoenix-core/src/main/java/org/apache/phoenix/iterate/RegionScannerFactory.java:
##########
@@ -285,8 +286,11 @@ public boolean nextRaw(List<Cell> result) throws 
IOException {
             }
           }
           Cell arrayElementCell = null;
-          if (arrayFuncRefs != null && arrayFuncRefs.length > 0 && 
arrayKVRefs.size() > 0) {
-            int arrayElementCellPosition = 
replaceArrayIndexElement(arrayKVRefs, arrayFuncRefs, result);
+          if (serverParsedFuncRefs != null && serverParsedFuncRefs.length > 0 
&& serverParsedKVRefs.size() > 0) {
+            int

Review Comment:
   actually I used IntelliJ formatting to avoid running longer than 100 or 150 
chars in a line. I can split into 2 line instead of 3. just went with the 
default formatting





> Parse JSON columns on the server side
> -------------------------------------
>
>                 Key: PHOENIX-7073
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-7073
>             Project: Phoenix
>          Issue Type: Sub-task
>            Reporter: Ranganath Govardhanagiri
>            Assignee: Ranganath Govardhanagiri
>            Priority: Major
>
> One of the optimizations mentioned in the Design spec is to _"pass the json 
> path to server side as part of the select and do the parsing on the server 
> side and then return the result to the client."_
> This is the task to track that change



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to