Commit 33d7461
committed
[fix](parser) Validate ORDER BY must come before LIMIT in SQL syntax
Add validation to ensure ORDER BY appears before LIMIT in SQL queries.
- Add two-layer detection in withQueryOrganization method
- Use LogicalSubQueryAlias as boundary marker for query levels
- Add 17 comprehensive test cases covering error and valid cases
- Fix incorrect syntax like 'SELECT * FROM t LIMIT 10 ORDER BY a'
- Allow valid multi-level queries with subqueries1 parent 4eba7f9 commit 33d7461
5,230 files changed
Lines changed: 57729 additions & 118380 deletions
File tree
- .claude/skills/code-review
- .github
- workflows
- be
- benchmark
- cmake
- src
- agent
- cloud
- common
- kerberos
- core
- exec
- common
- es
- operator
- join
- schema_scanner
- sink/writer/iceberg
- exprs
- aggregate
- geo
- glibc-compatibility/musl
- http
- action
- index-tools
- information_schema
- io
- cache
- fs
- hdfs
- tools
- olap
- rowset
- segment_v2
- ann_index
- cmake-protect
- inverted_index
- analyzer
- basic
- icu
- ik
- cfg
- core
- dic
- char_filter
- normalizer
- query_v2
- all_query
- bit_set_query
- boolean_query
- const_score_query
- phrase_prefix_query
- phrase_query
- postings
- prefix_query
- regexp_query
- term_query
- union
- wildcard_query
- query
- phrase_query
- similarity
- token_filter
- tokenizer
- basic
- char
- empty
- icu
- keyword
- ngram
- pinyin
- standard
- util
- variant
- task
- wal
- pch
- pipeline
- common
- data_gen_functions
- exec
- join
- local_exchange
- query_cache
- shuffle
- runtime_filter
- runtime
- cache
- index_policy
- memory
- routine_load
- stream_load
- workload_group
- workload_management
- service
- arrow_flight
- storage
- index/inverted
- schema_change
- segment/variant
- tools
- udf/python
- util
- arrow
- hash
- mustache
- simd
- vec
- aggregate_functions
- columns
- common
- hash_table
- sort
- string_utils
- core
- data_types
- serde
- exec
- executor
- time_sharing
- tools/simulator
- format
- arrow
- avro
- csv
- file_reader
- json
- native
- orc
- parquet
- table
- hive
- iceberg
- text
- wal
- scan
- exprs
- lambda_function
- table_function
- functions
- ai
- array
- cast
- url
- io
- jsonb
- json
- olap
- runtime
- sink
- writer
- iceberg
- maxcompute
- spill
- utils
- test
- agent
- ai
- cloud
- common
- core/column
- exec
- hash_map
- operator
- pipeline
- common
- schema_scanner
- exprs
- geo
- gutil/strings
- http
- io
- cache
- fs
- olap
- rowset
- segment_v2
- inverted_index
- analyzer
- ananlyzer
- char_filter
- common
- compaction
- util
- data
- query_v2
- boolean_query
- query
- phrase_query
- similarity
- token_filter
- tokenizer
- util
- mock
- test_data
- converter_test_data/data/0/15007/368169781
- incremental_delta
- pending_delta
- row_table
- vector_index
- vector_search
- wal
- pipeline
- common
- exec
- operator
- shuffle
- runtime_filter
- runtime
- cache
- index_policy
- memory
- plugin
- workload_group
- scan
- service
- storage
- index/inverted/query/phrase_query
- segment
- testutil
- mock
- udf/python
- util
- simd
- vec
- aggregate_functions
- columns
- common
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
51 | | - | |
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
90 | 90 | | |
91 | 91 | | |
92 | 92 | | |
93 | | - | |
94 | | - | |
95 | 93 | | |
96 | 94 | | |
97 | 95 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
90 | 90 | | |
91 | 91 | | |
92 | 92 | | |
93 | | - | |
94 | 93 | | |
95 | 94 | | |
96 | 95 | | |
| |||
This file was deleted.
0 commit comments