deniskuzZ commented on code in PR #4421:
URL: https://github.com/apache/hive/pull/4421#discussion_r1265584897
##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/CompactionTxnHandler.java:
##########
@@ -206,9 +207,11 @@ public Set<CompactionInfo> findPotentialCompactions(int
abortedThreshold,
") \"C\" " +
"ON \"TC\".\"CTC_DATABASE\" = \"C\".\"CC_DATABASE\" AND
\"TC\".\"CTC_TABLE\" = \"C\".\"CC_TABLE\" " +
" AND (\"TC\".\"CTC_PARTITION\" = \"C\".\"CC_PARTITION\" OR
(\"TC\".\"CTC_PARTITION\" IS NULL AND \"C\".\"CC_PARTITION\" IS NULL)) " +
- "WHERE \"C\".\"CC_ID\" IS NOT NULL OR " +
isWithinCheckInterval("\"TC\".\"CTC_TIMESTAMP\"", checkInterval) : "");
+ "WHERE \"C\".\"CC_ID\" IS NOT NULL OR " +
isWithinCheckInterval("\"TC\".\"CTC_TIMESTAMP\"", checkInterval) : ""));
LOG.debug("Going to execute query <{}>", s);
+ stmt.setFetchSize(fetchSize);
Review Comment:
what is the impact of using that flag?
https://stackoverflow.com/questions/1318354/what-does-statement-setfetchsizensize-method-really-do-in-sql-server-jdbc-driv/15559710#15559710
The fetchSize parameter is a hint to the JDBC driver as to many rows to
fetch in one go from the database. But the driver is free to ignore this and do
what it sees fit. Some drivers, like the Oracle one, fetch rows in chunks, so
you can read very large result sets without needing lots of memory.
--
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]