Skip to content

Support expressions in order_by#400

Merged
ea-rus merged 2 commits intostagingfrom
fix-order
Sep 18, 2024
Merged

Support expressions in order_by#400
ea-rus merged 2 commits intostagingfrom
fix-order

Conversation

@ea-rus
Copy link
Collaborator

@ea-rus ea-rus commented Sep 12, 2024

Parser:
Support expression in order by clause:

select *
from tbl
order by a*b -- <--

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
mindsdb_sql
   __about__.py10100%1–10
   __init__.py1292382%43, 47, 93, 110, 113, 134–153, 160–161, 178
mindsdb_sql/parser
   lexer.py165199%243
   logger.py19479%14, 17, 23, 26
   parser.py4563193%48, 53, 73, 77, 81, 87, 91–93, 121–128, 182, 199, 249, 253, 317, 346–347, 367, 378, 491, 526, 557, 567, 581, 657
   utils.py46491%73–79
mindsdb_sql/parser/ast
   base.py36781%13, 28, 31, 49–52
   create.py761087%7–8, 23, 28, 91–96
   drop.py52296%10, 13
   insert.py63494%39–41, 46
   show.py48198%18
   update.py53591%40–42, 75–76
mindsdb_sql/parser/ast/select
   case.py26292%19, 22
   constant.py36197%23
   data.py11282%15, 19
   identifier.py64789%43, 80–85
   native_query.py13192%25
   operation.py131497%56, 65, 168, 187
   parameter.py11191%10
   select.py97397%148–153
   star.py12283%8–9
   union.py23196%18
mindsdb_sql/parser/dialects/mindsdb
   knowledge_base.py46198%79
   lexer.py2622192%343, 345, 347, 359, 361, 363, 369–387
   parser.py10182697%119, 123, 262, 287, 384, 570, 587, 611–612, 808, 862, 939, 1068, 1078, 1116–1117, 1146, 1157, 1239, 1302, 1362, 1830, 1838, 1891–1894
mindsdb_sql/parser/dialects/mysql
   lexer.py23674%16, 18, 20, 32, 34, 36
   parser.py6092496%131, 133, 226–227, 326, 356, 373, 423, 427, 501, 511, 549–550, 570, 581, 702, 730, 735–737, 741, 758, 902, 1046
   show_index.py171135%10–12, 15–23, 26–27
mindsdb_sql/planner
   plan_join.py3221396%141, 179, 188, 215, 219, 308, 358, 376, 385–386, 435, 445, 447
   plan_join_ts.py1791094%53, 60, 83–84, 129, 181, 185, 193, 268, 335
   query_plan.py22864%13–21, 30
   query_planner.py4146784%41–42, 65–66, 75, 166, 309, 327–329, 336, 437, 465–543, 586–587, 600, 626, 688, 718, 746, 771–773
   query_prepare.py2846179%76, 78, 86–116, 227, 249, 272, 315, 336, 360, 389–443, 446–452, 483, 496–498, 507, 526
   step_result.py14286%7, 12
   steps.py1581690%12, 17, 25, 30–32, 51–53, 61–64, 90–92
   ts_utils.py67691%59, 61, 83–86, 90
   utils.py2422789%60, 106, 112, 127, 139, 151, 154, 166, 194, 207, 222, 228, 233, 240, 242, 247, 260, 265, 271, 277, 291, 308, 338–339, 350–351, 366
mindsdb_sql/render
   sqlalchemy_render.py4498082%56, 91, 93, 105, 110, 127, 129, 131, 133, 195, 198–199, 212–213, 224–225, 243, 273, 275, 310, 324, 330, 381, 444–455, 466–470, 500, 515, 520–530, 539–540, 543–544, 551, 574–585, 605, 638–662, 685, 691, 731–734, 755–758
TOTAL656050592% 

Tests Skipped Failures Errors Time
680 1 💤 0 ❌ 0 🔥 24.943s ⏱️

@ea-rus ea-rus mentioned this pull request Sep 18, 2024
@ea-rus ea-rus merged commit 4c63835 into staging Sep 18, 2024
@ea-rus ea-rus mentioned this pull request Oct 31, 2024
@ea-rus ea-rus deleted the fix-order branch October 31, 2024 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: merged

Development

Successfully merging this pull request may close these issues.

1 participant