[ZOOKEEPER-1325] Log maxClientCnxn warning in INFO level

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Invalid
• Affects Version/s: 3.3.3, 3.3.4, 3.4.0
• Component/s: server
• Labels: noob
• Assignee: Unassigned
• Reporter: Mubarak Seyed
• Created: 09/Dec/11 22:09
• Updated: 09/Dec/11 23:36
• Resolved: 09/Dec/11 22:30

Description

When Hbase client ZooKeeperWatcher gets ConnectionLossException (/hbase/rs or /hbase), it is very hard debug the ZK server log if ZK server has started using log4j INFO level.
When maxClientCnxn limit is reached for a single client (at the socket level), it will be nice to log it in INFO level instead of WARN. It will help hbase clients (Region server, HMaster, and HBase cient lib) to debug the issue in production.

3.4 - src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java 
3.3.4 - src/java/main/org/apache/zookeeper/server/NIOServerCnxn.java

 public void run() {
        while (!ss.socket().isClosed()) {
            try {
                        ...
                        ...
                
                        if (maxClientCnxns > 0 && cnxncount >= maxClientCnxns){
                            LOG.info("Too many connections from " + ia
                                     + " - max is " + maxClientCnxns );
                            sc.close();
                        } 
                        ... 
    }