Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-1578

Connector constructor doesn't need to fail-fast (maybe?)

    XMLWordPrintableJSON

Details

    • Task
    • Status: Resolved
    • Major
    • Resolution: Won't Fix
    • None
    • None
    • client
    • None

    Description

      Currently, when one instantiates a Connector from an Instance, with one's username and token, the Connector actually tries to reach out and authenticate those credentials, to provide early failure.

      Because this has some overhead, we explicitly check for the condition that the user is a system user (a tserver or the monitor, etc.), so that we don't fail early in these conditions (because we don't expect the system to be poorly configured or running in a bad state, such that it doesn't authenticate... and we don't care about causing an inconvenience to unauthorized servers by failing late).

      After some thought, I'm not sure that we should continue to fail fast. I'm not sure that it provides sufficient convenience for users to warrant the additional RPC calls. Besides, there's no guarantee, especially with the new pluggable authentication introduced in 1.5.0, that the credentials will still be valid later, even if the user were able to authenticate initially.

      As such, I propose we drop this check, and rely on authentication failures later (when work is actually initiated).

      Attachments

        Issue Links

          Activity

            People

              ctubbsii Christopher Tubbs
              ctubbsii Christopher Tubbs
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - 0.25h
                  0.25h
                  Remaining:
                  Remaining Estimate - 0.25h
                  0.25h
                  Logged:
                  Time Spent - Not Specified
                  Not Specified