Preserve Comments in SQL Query
I would like to follow up this post regarding preserving comments while editing a query in the QueryEditor. There it is stated that this will be implemented in the next major release. Is this still valid?
Replies 11
In Oracle, optimizer hints are realized as comments. You may take a look at this documentation. If I didn't miss anything, then it would only require some extensions to the existing SQL parser and no rework. The context of optimizer hints (i.e. to which item of the query they belong) is clearly specified.
At least this special kind of comment is indispensable for us. They reduce execution time by orders of magnitudes.
However, the specification shows a pattern about how to assign comments to an item in the SQL statement. From that pattern you could derive a general approach to assign comments to a part of a SQL statement on a best-effort basis.
Finally, even if you at least collect all comments at one place and render them at the beginning/end of the actual statement would already help us. Everything would be better then just forgetting them. :)
Thank you for sharing your thoughts. We will think once again about the possiblity to implement this feature.
Oracle hints are preserved right now.
We have another approach that we assume is less invasive and easier to implement for you:
Add a "Comment" text-column in the GridView for editing the (sub-)query columns. Store the value of this comment in the LayoutXml.
That could be an intermediate step that would already help us a lot.
Next step could be to parse/render this from/to the final SQL as well, as discussed above.
What do you think?
It's not a problem of finding the place in UI to show comments. The main part of this task is to parse and then insert comments in result SQL query text. This task will need the time that we can't afford to spend now.
It is possible to add a custom column to the QueryColumnListControl, there's a special demo project that illustrates this feature. For now it is not possible to save comments in layout XML, but we'll think about the best way to achieve this.
Hello, Christian.
I must admit that we haven't done this feature so far. I am terribly sorry for the inconvenience this cause everybody.
The major SQL parser update is delayed. Hopefully, we will get back to this feature in the second half of this year, but I can't promise you this.
The implementation of this functionality requires significant rework the parsing mechanism of our component. So we postponed its implementation for higher-priority tasks. I believe that we'll get back to it in the future, but we have no clear timeline for the task.