startreedata / pinot-client-go / 16060799801 / 1
94%
master: 94%

DEFAULT BRANCH: master
Ran
Files 11
Run time 0s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 93.557% (+1.3%) from 92.279%
16060799801.1

push

github

web-flow 
Add PreparedStatement Support - Fixes Issue #22 (#46)

* Add PreparedStatement support for Issue #22

- Implement PreparedStatement interface with thread-safe parameter management
- Add Connection.Prepare() method for creating prepared statements
- Support for multiple parameter types (string, int, int64, float64, bool, interface{})
- Provide Execute() and ExecuteWithParams() methods for query execution
- Include comprehensive unit tests with mock servers and concurrency tests
- Add integration tests for real Pinot cluster validation
- Create example application demonstrating PreparedStatement usage patterns
- Support for both regular and multistage query execution engines

Features:
- Thread-safe parameter setting and execution
- 1-based parameter indexing (like Java PreparedStatement)
- Proper resource management with Close() method
- Parameter validation and error handling
- Reuse of existing formatArg() function for type handling
- Clear parameters and query metadata access

This implementation provides PreparedStatement functionality equivalent to
the Java client while following Go idioms and best practices.

* Fix test issues and improve test coverage

- Fix malformed JSON in PreparedStatement mock server test
- Remove unused imports in PreparedStatement example
- All PreparedStatement tests now pass with excellent coverage:
  - Overall coverage: 91.4% of statements
  - PreparedStatement functions: 80-100% coverage each
  - All core PreparedStatement methods have 100% coverage
  - Execute methods have 90%+ coverage
  - Comprehensive test suite including mock servers, concurrency, and edge cases

* Fix linter issues for CI compliance

- Remove redundant newline in fmt.Println statement
- Fix unused parameter 'r' in HTTP handler by changing to '_'
- Add proper error handling for defer stmt.Close() calls in tests
- Use defer func() { _ = stmt.Close() }() pattern to explicitly ignore errors in test cleanup
- All linter checks now pass: gofmt, errche... (continued)

668 of 714 relevant lines covered (93.56%)

6.61 hits per line

Source Files on job unit - 16060799801.1