[PR] feat(cli): add Fluss CLI module with comprehensive test coverage [fluss]

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

gnuhpc opened a new pull request, #2356:
URL: https://github.com/apache/fluss/pull/2356

   ## Summary
   Add a new `fluss-cli` module providing interactive SQL command-line interface for Fluss clusters.
   
   ## Core Features
   - Interactive SQL REPL shell for Fluss clusters
   - Command-line SQL execution with -e and -f options
   - ASCII table formatting for query results
   - Complex type support (ARRAY, MAP, ROW)
   - Connection configuration management
   - Warning suppression for clean output (FLUSS_CLI_SUPPRESS_WARNINGS)
   
   ## Test Coverage
   - **114 unit tests** covering all core functionality
   - Integration test script (fluss-cli-release-check.sh) with **35 test cases**
   - All tests passing with **0 failures**
   - Checkstyle: **0 violations**
   - Apache RAT: all files licensed
   
   ## Integration
   - Add fluss-cli module to root pom.xml
   - Package CLI JAR in fluss-dist distribution
   - Add CLI documentation to website
   
   ## Files Changed
   - **29 files changed, 6,505 insertions(+)**
   - New module: fluss-cli/ with complete source, tests, and documentation


-- 
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: issues-unsubscribe@fluss.apache.org

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

Re: [PR] feat(cli): add Fluss CLI with AST-based parsing and 71% test coverage [fluss]

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

gnuhpc commented on PR #2356:
URL: https://github.com/apache/fluss/pull/2356#issuecomment-3764040264

   ## Update: Squashed Commits + WHERE Filtering
   
   I've force-pushed a clean, squashed commit that includes:
   
   ### Changes in This Update
   1. **Squashed 6 commits → 1 clean commit** for easier review
   2. **Added client-side WHERE filtering** for Log table queries
   3. **Smart column fetching** (SELECT cols ∪ WHERE cols) to minimize network transfer
   4. **10 new filtering tests** (all passing)
   
   ### WHERE Filtering Implementation
   - **Supported operators**: `=`, `<>`, `>`, `<`, `>=`, `<=`, `AND`, `OR`
   - **Architecture**: Server-side projection + client-side filtering
   - **Example flow**:
     ```sql
     SELECT name FROM events WHERE age > 25 AND status = 'active'
     ```
     1. Extract WHERE columns: `[age, status]`
     2. Fetch columns: `[name, age, status]` (optimized)
     3. Server projects these 3 columns
     4. Client filters: `age > 25 AND status = 'active'`
     5. Client projects to: `[name]`
     6. Display: name only
   
   ### Test Results
   - **Total tests**: 201 (10 new filtering tests)
   - **Coverage**: 70.4% instruction, 58.2% line
   - **All tests passing**: ✅
   
   ### Commit Hash
   - **Before**: `95a8e2d0` (6 commits)
   - **After**: `d7a8dd67` (1 squashed commit)


-- 
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: issues-unsubscribe@fluss.apache.org

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

Re: [PR] feat(cli): add Fluss CLI with AST-based parsing and 71% test coverage [fluss]

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

polyzos commented on PR #2356:
URL: https://github.com/apache/fluss/pull/2356#issuecomment-6251759657

   @gnuhpc  whats the status here? should we close this or is it WIP?


-- 
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: issues-unsubscribe@fluss.apache.org

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