search

PILLUTLAAVINASH / google-enterprise-connector-manager

Automatically exported from code.google.com/p/google-enterprise-connector-manager
0 stars 0 forks source link

DocPusher uses a logger named WorkQueue #45

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. Read the source.

What is the expected output? What do you see instead?

The logger is initialized using WorkQueue.class.getName(). I suspect this
is just a copy-and-pasto, and that the logger should be initialized using
DocPusher rather than WorkQueue.

What version of the product are you using? On what operating system?

Connector Manager revision 361.

Please provide any additional information below.

Original issue reported on code.google.com by jl1615@gmail.com on 18 May 2007 at 10:36

GoogleCodeExporter commented 8 years ago 
In the same vein, the EncryptedPropertyPlaceholderConfigurer logged is 
initialized using 
InstanceInfo.class.getName().

Original comment by jl1615@gmail.com on 7 Dec 2007 at 11:24

GoogleCodeExporter commented 8 years ago 
Fixed trunk/r736

r736 | mgronber | 2007-12-19 17:12:18 -0800 (Wed, 19 Dec 2007) | 5 lines

Fix Issue 45, change DocPusher logging methods to be the same, added
one additional debug statement to the DocPusher, and made change to
QueryTraverser log statement to correctly use the interface rather
than the Impl.

Original comment by mgron...@gmail.com on 20 Dec 2007 at 1:14

GoogleCodeExporter commented 8 years ago 
Although the DocPusher logger is fixed, there are other logger names that might 
be broken:
key:  ClassName:LoggerName

EncryptedPropertyPlaceholderConfigurer: InstanceInfo
AuthorizationParser: AuthorizationHandler
JcrTraversalManagerTest: MockRepositoryPropertyTest
MockJcrNodeIteratorTest: MockRepositoryPropertyTest
MockJcrNodeTest: MockRepositoryPropertyTest
MockJcrQueryManagerTest: MockRepositoryPropertyTest
MockRepositoryPropertyListTest: MockRepositoryPropertyTest
ConnectorTestUtilsTest: MockRepositoryPropertyTest

Original comment by Brett.Mi...@gmail.com on 22 May 2008 at 3:15

GoogleCodeExporter commented 8 years ago 
The above additional Logger names were fixed by me in revision r810 .

Original comment by Brett.Mi...@gmail.com on 30 May 2008 at 6:18

GoogleCodeExporter commented 8 years ago

Original comment by mgron...@gmail.com on 19 Jun 2008 at 6:35