babelfish-for-postgresql / babelfish_extensions

Babelfish for PostgreSQL provides the capability for PostgreSQL to work with applications written for Microsoft SQL Server. Babelfish understands the SQL Server wire-protocol and T-SQL, the Microsoft SQL Server query and procedural language, so you don’t have to switch database drivers or rewrite all of your application queries.
https://babelfishpg.org/
Apache License 2.0
274 stars 93 forks source link

Revert "Add ANTLR parsing time to EXPLAIN info (#2597)" #2631

Closed Deepesh125 closed 4 months ago

Deepesh125 commented 4 months ago

This reverts commit 4330bab8492e17645e5dc2081ee30d1ea4132164.

Commit details of 4330bab8492e17645e5dc2081ee30d1ea4132164 -

Add ANTLR parsing time to EXPLAIN info (#2597)

This commit adds support to show ANTLR parsing time info with EXPLAIN info. With this change, query plan would look like following:
```
QUERY PLAN

----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Query Text: select 'abc'

Result  (cost=0.00..0.01 rows=1 width=32) (actual time=0.004..0.005 rows=1 loops=1)

ANTLR Parsing Time: 34.469 ms

Planning Time: 0.016 ms

Execution Time: 0.057 ms
```
This capabilities will help us diagnosing the performance issue to understand where most of the time being consumed.

Task: BABEL-3650 Signed-off-by: Dipesh Dhameliya dddhamel@amazon.com

Check List

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.

For more information on following Developer Certificate of Origin and signing off your commits, please check here.

coveralls commented 4 months ago

Pull Request Test Coverage Report for Build 9284059167

Details


Totals Coverage Status
Change from base Build 9274688228: 0.001%
Covered Lines: 41909
Relevant Lines: 57487

💛 - Coveralls