[GitHub] [ignite] liyuj commented on pull request #10671: IGNITE-19378 Ignite Docs: Remove docs related to lazy loading

["liyuj (via GitHub)" <gi...@apache.org>]

liyuj commented on PR #10671:
URL: https://github.com/apache/ignite/pull/10671#issuecomment-1537309948

   Thanks for your review!
   
   The PR is improved, please check again.
   
   在 2023/5/7 00:44, Aleksey Plekhanov 写道:
   >
   > ***@***.**** commented on this pull request.
   >
   > ------------------------------------------------------------------------
   >
   > In docs/_docs/perf-and-troubleshooting/sql-tuning.adoc 
   > <https://github.com/apache/ignite/pull/10671#discussion_r1182812329>:
   >
   > > @@ -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
   >
   > 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.
   >
   > —
   > Reply to this email directly, view it on GitHub 
   > <https://github.com/apache/ignite/pull/10671#pullrequestreview-1409509785>, 
   > or unsubscribe 
   > <https://github.com/notifications/unsubscribe-auth/ACGRBAQUDW3YBDTAFVWGZ4TXEZ5VDANCNFSM6AAAAAAXOSGD6M>.
   > You are receiving this because you authored the thread.Message ID: 
   > ***@***.***>
   >


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