PakhomovAlexander opened a new pull request, #7499:
URL: https://github.com/apache/ignite-3/pull/7499

   ## Summary
   
   Implements row limiting for SQL query results in the CLI to prevent 
OutOfMemoryError when querying large tables.
   
   - Add `--limit` option to `sql` command (default: 1000 rows)
   - Add `ignite.cli.sql.result-limit` config setting
   - Display truncation indicator when results are limited: `... (showing first 
N rows, more available)`
   - `--limit=0` disables limiting and returns all rows
   
   ## Changes
   
   - **Table.java**: Add `fromResultSet(ResultSet, int limit)` method with 
`hasMoreRows` flag
   - **SqlQueryResultTable.java**: Add truncation indicator rendering
   - **SqlCommand.java/SqlReplCommand.java**: Add `--limit` option
   - **CliConfigKeys.java**: Add `SQL_RESULT_LIMIT` config key
   - **TableTest.java**: Unit tests for limit functionality
   - **ItSqlRowLimitTest.java**: Integration tests for `--limit` option
   
   ## Test plan
   
   - [x] Unit tests for `Table.fromResultSet()` with various limit scenarios
   - [x] Integration tests for `--limit` CLI option
   - [ ] Manual testing with large datasets to verify OOM prevention
   - [ ] Verify truncation indicator displays correctly
   
   https://issues.apache.org/jira/browse/IGNITE-27516
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


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

Reply via email to