summaryrefslogtreecommitdiff
path: root/connectivity/source/drivers/jdbc/jservices.cxx
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2012-01-13 09:38:07 +0100
committerStephan Bergmann <sbergman@redhat.com>2012-01-13 09:43:26 +0100
commitbb59742bcf4883af5876a2ffadcc4a689e414b60 (patch)
tree32083c35eb4968f9c817f084ede88f26e753c025 /connectivity/source/drivers/jdbc/jservices.cxx
parent1b8b6ecf5fdf753bd787748d54aaa61964199465 (diff)
Confine JDBC driver to thread-affine apartment for Java 6 performance
... so that the massive number of the JDBC driver's calls to JNI Attach/Detach- CurrentThread are guaranteed not to happen on the main thread (where they are extremely expensive, see <http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6978641> "Fix for 6929067 introduces additional overhead in thread creation/termination paths").
Diffstat (limited to 'connectivity/source/drivers/jdbc/jservices.cxx')
-rw-r--r--connectivity/source/drivers/jdbc/jservices.cxx16
1 files changed, 15 insertions, 1 deletions
diff --git a/connectivity/source/drivers/jdbc/jservices.cxx b/connectivity/source/drivers/jdbc/jservices.cxx
index 59c73ff6cbc5..85df77190783 100644
--- a/connectivity/source/drivers/jdbc/jservices.cxx
+++ b/connectivity/source/drivers/jdbc/jservices.cxx
@@ -83,7 +83,21 @@ struct ProviderRequest
void* getProvider() const { return xRet.get(); }
};
-//---------------------------------------------------------------------------------------
+extern "C" SAL_DLLPUBLIC_EXPORT void SAL_CALL
+component_getImplementationEnvironment(
+ char const ** ppEnvTypeName, uno_Environment **)
+{
+ // Recent Java 6 VMs make calls to JNI Attach/DetachCurrentThread (which
+ // this code does extensively) very expensive. A follow-up JVM fix reduced
+ // the overhead significantly again for all threads but the main thread. So
+ // a quick hack to improve performance of this component again is to confine
+ // it in the affine apartment (where all code will run on a single,
+ // dedicated thread that is guaranteed no to be the main thread). However,
+ // a better fix would still be to redesign the code so that it does not call
+ // Attach/DetachCurrentThread so frequently:
+ *ppEnvTypeName = CPPU_CURRENT_LANGUAGE_BINDING_NAME ":affine";
+}
+
extern "C" SAL_DLLPUBLIC_EXPORT void* SAL_CALL component_getFactory(
const sal_Char* pImplementationName,
void* pServiceManager,