PILLUTLAAVINASH / google-enterprise-connector-manager

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

LoginException class and ResultSet interface names clash with Java SE class names #19

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?

See below.

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

The LoginException class and ResultSet interface names clash with Java SE
class names. It's hard to avoid all conflicts, but I would suggest
PropertyMapList in place of ResultSet, since you've already introduced the
use of "Map" rather than inventing a new term (like Node or Item).
TraversalResults is my runner-up suggestion.

My only two thoughts for LoginException were AuthenticationException, which
is also taken, or just dropping it. Dropping it doesn't seem heinous. As
I've mentioned, it can be difficult to tell authentication errors from
other errors when talking to a remote repository. Also, the connector
manager does not seem to be doing anything profound with the distinction,
other than logging it, which is probably not much more helpful than the
error message itself.

Please use labels and text to provide additional information.

Original issue reported on code.google.com by donald.z...@gmail.com on 24 Jan 2007 at 3:35

GoogleCodeExporter commented 8 years ago
Google Bug #243991

Original comment by vjo...@gmail.com on 9 Feb 2007 at 4:30

GoogleCodeExporter commented 8 years ago
LoginException renamed as RepositoryLoginException in R415

Other renaming done in R431.

Original comment by mgron...@gmail.com on 3 Oct 2007 at 9:57

GoogleCodeExporter commented 8 years ago

Original comment by mgron...@gmail.com on 3 Oct 2007 at 10:51