search

tanelpoder / tpt-oracle

Tanel Poder's Performance & Troubleshooting Tools for Oracle Databases
https://tanelpoder.com
Other
657 stars 312 forks source link

Changed column headers, fixes #12, #13 #14

Closed nklamann closed 2 years ago

nklamann commented 5 years ago

Forgot one file

tanelpoder commented 5 years ago

FOR A10 won't work for numeric columns in sqlplus. Looks like SQL Developer allows that, but this needs to work correctly on sqlplus too (so, "FORMAT 99999...."-like syntax is needed for numeric columns). Please change that in your PR.

SQL> col x for a15
SQL> select 123456 x from dual;

         X
----------
##########

Two more comments: 1) I didn't even realize that I had a sql_id.sql .... looks like an old copy of sqlid.sql that I'm using by default. I'll remove sql_id.sql as it's a stale copy (and sqlid.sql allows to type less!)

2) I've come to a realization that asqlmon.sql output is too wide, it doesn't need the wide columns in the end (object alias/predicates). The reality is that plenty of people still use TPT scripts in terminals like putty that can't horizontally scroll. So thinking of commenting out these 2 columns in asqlmon.sql and having asqlmonx.sql (as I tend to have eXtended versions of scripts)