NodeToString of CursorNode does not include OFFSET, LIMIT
Bug #1183917 reported by
Nathan Williams
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Akiban SQL Parser |
Fix Released
|
Wishlist
|
Mike McMahon | ||
Bug Description
Parser r300
Only the resultSetNode and orderByList are included. Is there a pattern dictating what should be included or are elements added as they are encountered?
If the latter, including the offset and fetchFirst nodes would be helpful.
Related branches
lp:~mmcm/akiban-sql-parser/misc-non-akserver
- Thomas Jones-Low: Needs Fixing
- Nathan Williams: Approve
-
Diff: 789 lines (+454/-21)17 files modifiedsrc/main/java/com/akiban/sql/parser/AggregateWindowFunctionNode.java (+4/-0)
src/main/java/com/akiban/sql/parser/BinaryRelationalOperatorNode.java (+8/-8)
src/main/java/com/akiban/sql/parser/FullOuterJoinNode.java (+76/-0)
src/main/java/com/akiban/sql/parser/NodeFactoryImpl.java (+9/-0)
src/main/java/com/akiban/sql/parser/NodeNames.java (+6/-0)
src/main/java/com/akiban/sql/parser/NodeTypes.java (+4/-1)
src/main/java/com/akiban/sql/parser/PartitionByColumn.java (+85/-0)
src/main/java/com/akiban/sql/parser/PartitionByList.java (+27/-0)
src/main/java/com/akiban/sql/parser/WindowDefinitionNode.java (+33/-8)
src/main/java/com/akiban/sql/unparser/NodeToString.java (+73/-0)
src/main/javacc/SQLGrammar.jj (+69/-4)
src/test/resources/com/akiban/sql/parser/select-7.expected (+55/-0)
src/test/resources/com/akiban/sql/parser/select-7.sql (+1/-0)
src/test/resources/com/akiban/sql/unparser/select-17.expected (+1/-0)
src/test/resources/com/akiban/sql/unparser/select-17.sql (+1/-0)
src/test/resources/com/akiban/sql/unparser/select-18.expected (+1/-0)
src/test/resources/com/akiban/sql/unparser/select-18.sql (+1/-0)
| Changed in akiban-sql-parser: | |
| status: | New → Fix Committed |
| assignee: | nobody → Mike McMahon (mmcm) |
| milestone: | none → 1.0.16 |
| Changed in akiban-sql-parser: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
