[
https://issues.apache.org/jira/browse/PHOENIX-6711?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17537279#comment-17537279
]
ASF GitHub Bot commented on PHOENIX-6711:
-----------------------------------------
chrajeshbabu commented on code in PR #1437:
URL: https://github.com/apache/phoenix/pull/1437#discussion_r873246623
##########
phoenix-core/src/main/java/org/apache/phoenix/iterate/TableResultIterator.java:
##########
@@ -143,6 +143,20 @@ public TableResultIterator(MutationState mutationState,
Scan scan, ScanMetricsHo
ScanUtil.setScanAttributesForClient(scan, table,
plan.getContext().getConnection());
}
+ public TableResultIterator(MutationState mutationState, Scan scan,
ScanMetricsHolder scanMetricsHolder,
Review Comment:
@gjacoby126 @joshelser
Actually I thought of adding the special constructor to use in phoenix spark
connector to avoid fetching the metadata and avoid using the query plan but
it's difficult to avoid using the query plan because we need the row projector,
query context etc.. So I have found a simple solution of sharing the table meta
data across the worker and cache it so that we need not fetch the metadata
everytime in the workers and avoid the unnecessary load on RS holding the
system catalog table. Here is the PR created under phoenix-connectors
https://github.com/apache/phoenix-connectors/pull/80 with that changes in place
this special constructor not required so will remove these special constructor
changes and update the PR.
> Add support of skipping the system tables existence check during connection
> initialisation and create new table result iterator which doesn't require
> fetch meta data of table
> ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: PHOENIX-6711
> URL: https://issues.apache.org/jira/browse/PHOENIX-6711
> Project: Phoenix
> Issue Type: Sub-task
> Reporter: Rajeshbabu Chintaguntla
> Assignee: Rajeshbabu Chintaguntla
> Priority: Major
> Fix For: 5.1.2
>
>
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
