Trafodion

Clone Tools
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Fix to ensure that trafci returns the correct status message of the recent sql command instead of returning "--- SQL operation complete" always.

Optimizing and avoiding unnecessary function calls shown as hot spots in jprofile - part-IIa

Removed ununsed code

Fix for cores seen with T2 JDBC applications caused by PR 1820

New member variables were not initialized in the CliGlobals constructor

Optimizing and avoiding unnecessary function calls shown as hot spots in jprofile - part-II

Changed Hashtable to HashMap where possible with the assumption that the JDBC connection

shouldn't be used from different threads. TrafT4Connection object needs be made thread-safe.

Optimized org.trafodion.jdbc.t4.TrafT4ResultSet.findColumn by using HashMap of columnName to

columnIndex. This should avoid excessive calls to java.lang.String.equalsIgnoreCase.

Removed excessive calls to java.lang.Class.getMethod from TrafT4ResultSet

Rework addressing Qifan's suggestions

Cleaned up and ensured that JDBC driver conforms to transaction handling via Connection.commit and Connection.rollback. The transaction is always started by the database engine when it needs the transaction.

However, begin work and commit work is still supported via SQL command but

there is no transaction state is maintained in the driver for these commands.

Optimizing and avoiding unnecessary function calls shown as hot spots in jprofile - part-II

java.lang.String.split and javal.lang.String.replaceAll

JDBC driver was parsing the sql string passed to it by the application to determine the type of

SQL statement and to check if the sql string has parameters. It doesn't parse the sql string anymore

and passes the sql string to SQL engine via mxosrvr. In case of PreparedStatement, the JDBC driver will

set the sqlStmtType based on the query type returned. In case of Statement, the JDBC driver will set the

sqlStmtType as SQL_TYPE_UNKNOWN.

Now, the PreparedStatement is always prepared for setting array values for parameter for all

statement types such as IUD and select. SQL engine will report an error if the array values are

not supported for any statement type.

Optimizing and avoiding unnecessary function calls shown as hot spots in jprofile

java.net.Socket.setSoTimeout

org.trafodion.jdbc.t4.T4LoggingUtilities.log

java.net.Socket.getLocalSocketAddress

Now socket.read waits for the network timeout. It can be configured via a property networkTimeout or

call to connection.setNetworkTimeout. If the networkTimeout is not configured, a default value of 10 secs

is assumed.

The caller is expected to set the timeout depending upon the operation being performed like loginTimeout,

queryTimeout. Socket.read will wake up for every networkTimeout expiration and check if the timeout for the

operation hasn't exceeded. When it exceeds, SocketTimeoutException is thrown.

Changed the T4Statement and T4Restult classes to have "has a" relationship with T4Connection for

better understanding of the code. Earlier, it had "is a" relationship with T4Connection.

[TRAFODION-3290] Give better errors for case where table is all LOB columns

    • -25
    • +84
    /core/sql/regress/compGeneral/EXPECTED023
    • -0
    • +15
    /core/sql/regress/compGeneral/TEST023
Merge pull request #1820 from selvaganesang/trafodio_3280_2

[TRAFODION-3280] Reduce path length in Trafodion for improved performance and scalability

Changes as per review comments of PR 1820

[TRAFODION-3291] Fix core when multi-column stats are done on lots of columns

Merge pull request #1818 from SuJinpei/M-9933

[TRAFODION-3262] Insert float to numeric(19,0) lost precision for some case

Merge branch 'master' of github.com:apache/trafodion into trafodio_3280_2

[TRAFODION-3280] Reduce path length in Trafodion for improved performance and scalability

This PR fixes the following issues to improve performance.

1. When the session is dropped for the default context, we were unnecessarily

dropping HBase connections.

2. There were 2 HBase connections per mxosrvr - one for the default context and another

for repository context. Now it has been changed to one HBase connection per process.

    • -13
    • +12
    /core/sql/executor/HBaseClient_JNI.cpp
[TRAODION-3280] Reduce path length in Trafodion for improved performance and scalability

cli_globals::currContext() is now optimized for default context.

[TRAODION-3280] Reduce path length in Trafodion for improved performance and scalability

Changes to ensure that logging related objects are created and the corresponding

functions are called only when the logging Level is set higher than the recording level

Merge PR 1819

[TRAFODION-3292] Add descriptions and examples for the Identity Column in TSRFM

Merge PR 1817

Incorporate comments 1

    • -75
    • +31
    /win-odbc64/odbcclient/drvr35/drvrglobal.cpp
[TRAFODION-3289] Add the OVERLAY Function in the Trafodion SQL Reference Manual

PR-1816 TRAFODION-3288 trafici queries with multiple params return 'invalid desc' error

Merge PR 1814

Incorporate comments 2

Merge pull request #1815 from svarnau/j3281

[TRAFODION-3281] Fix syntax error introduced in previous change

fixes to trafci prepare/execute queries

- an extra msg 'pmd is null' no longer shows from trafci prepare execute

- trafci queries with multiple param no longer returns an 'invalid desc index' error

- changed reference to 'jave.sql.Varbinary' to 'java.sql.Varbinary'

[TRAFODION-3281] Fix syntax error introduced in previous change

Earlier changes for this jira introduced syntax error in script

with spaces, where tabs are expected.

    • -13
    • +13
    /core/sqf/sql/scripts/install_local_hadoop
Incorporate comments 1