You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2017/08/22 09:30:24 UTC

[directory-ldap-api] 01/01: Backported a fix : we were checking if the parent schema was loaded instead of the dependency, leading to useless CPU being consumed

This is an automated email from the ASF dual-hosted git repository.

elecharny pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git

commit 4d81716cadcb75955337d772ca41b6b81bd822c8
Author: Emmanuel Lécharny <el...@symas.com>
AuthorDate: Tue Aug 22 11:30:15 2017 +0200

    Backported a fix : we were checking if the parent schema was loaded instead of the dependency, leading to useless CPU being consumed
---
 .../directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java
index cff164d..d3f6026 100644
--- a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java
+++ b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java
@@ -1321,7 +1321,7 @@ public class DefaultSchemaManager implements SchemaManager
          */
         for ( String depName : deps )
         {
-            if ( registries.isSchemaLoaded( schemaName ) )
+            if ( registries.isSchemaLoaded( depName ) )
             {
                 // The schema is already loaded. Loop on the next schema
                 continue;

-- 
To stop receiving notification emails like this one, please contact
"commits@directory.apache.org" <co...@directory.apache.org>.