Skip to content

Conversation

@LissaGreense
Copy link
Owner

PR content:

  • Simplified nested conditions
  • Improve comments and messages
  • Introduced new SelectProcessor struct for better management of table transformations when generating select response s
  • Removed redundant return conditions
  • Introduced Columns types
  • Simplified interface casting for generic values

LissaGreense and others added 25 commits May 16, 2024 20:54
* Parser error handling wip

* Add error handling for engine and finish parser

---------

Co-authored-by: LissaGreense <walissa13@gmail.com>
JOIN FEATURE

* Add full join implementation

* Add full join error handling tests in engine

* Add documentation, lexer, parser and only tests for engine, handling full,inner,left,right joins

* Add enginge implemenatation and e2e tests for full,inner,left and right joins

* Refactore join method
* Add Aggregate function handling in lexer, parser and even write documentation

* Aggregate functions engine WIP

* Finish tests, fix implementation of aggr functions

---------

Co-authored-by: ixior462 <pawel.krupski@mail.com>
* Parser, lexer, token and ast impl

* engine, parser err handling, readme, tests for IN
* Tokens have been added

* Parser and lexer logic has been updated.

* Introduced a new section in README 'supported types'

* Add engine integration with NULL values

---------

Co-authored-by: LissaGreense <walissa13@gmail.com>
* Refactore e2e tests structure

* Update README in E2E section

* Move e2e test to seperate file
* Add apostrophe validation errors

* refactor

* Rewrite getExpression logic
* Upgrade go version and fix issues from Go Report Card

* Fix errorhandling tests

* Resolve go report card feed

* Refactore engine functions

* Change getSelectResponse function

* Refactor select response logic

* Improve comments

---------

Co-authored-by: ixior462 <pawel.krupski@mail.com>
@ixior462 ixior462 merged commit 306c2ca into main May 23, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants