You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ignite.apache.org by "alex-plekhanov (via GitHub)" <gi...@apache.org> on 2023/05/06 16:44:03 UTC

[GitHub] [ignite] alex-plekhanov commented on a diff in pull request #10671: IGNITE-19378 Ignite Docs: Remove docs related to lazy loading

alex-plekhanov commented on code in PR #10671:
URL: https://github.com/apache/ignite/pull/10671#discussion_r1182812329


##########
docs/_docs/perf-and-troubleshooting/sql-tuning.adoc:
##########
@@ -76,47 +76,6 @@ SELECT name FROM Person WHERE gender='M' and age = 30
 Avoid having too many columns in the result set of a `SELECT` query. Due to limitations of the H2 query parser, queries
 with 100+ columns may perform worse than expected.
 
-== Lazy Loading
-
-By default, Ignite attempts to load the whole result set to memory and send it back to the query initiator (which is

Review Comment:
   I'm not sure we should remove part related to the lazy loading completely. Flag deprecated but still in Ignite and can be used. Perhaps we should change description about default value and add warning about deprecation.



-- 
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: notifications-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org