Accumulo
  1. Accumulo
  2. ACCUMULO-2733

Credentials is missing deserializer to match Thrift serializer

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6.0
    • Component/s: None
    • Labels:
      None

      Description

      Right now Credentials has a toThrift method that serializes to a TCredential, but no corresponding fromThrift to deserialize. Instead there are several locations that inline this deserialization.

      To avoid maintaining multiple implementations, these should be consolidated into a single fromThrift method.

        Issue Links

          Activity

          Hide
          ASF subversion and git services added a comment -

          Commit e450d74aec76b751a3c2bc9ff12cd7da547f14f1 in accumulo's branch refs/heads/master from Sean Busbey
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=e450d74 ]

          ACCUMULO-2733 clean up one missed unused import.

          Show
          ASF subversion and git services added a comment - Commit e450d74aec76b751a3c2bc9ff12cd7da547f14f1 in accumulo's branch refs/heads/master from Sean Busbey [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=e450d74 ] ACCUMULO-2733 clean up one missed unused import.
          Hide
          ASF subversion and git services added a comment -

          Commit e450d74aec76b751a3c2bc9ff12cd7da547f14f1 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Sean Busbey
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=e450d74 ]

          ACCUMULO-2733 clean up one missed unused import.

          Show
          ASF subversion and git services added a comment - Commit e450d74aec76b751a3c2bc9ff12cd7da547f14f1 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Sean Busbey [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=e450d74 ] ACCUMULO-2733 clean up one missed unused import.
          Hide
          ASF subversion and git services added a comment -

          Commit 6742627f20ad58e8e0737baddde952d9d7edba87 in accumulo's branch refs/heads/master from Sean Busbey
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=6742627 ]

          ACCUMULO-2733 adds thrift deserialization to Credentials

          Show
          ASF subversion and git services added a comment - Commit 6742627f20ad58e8e0737baddde952d9d7edba87 in accumulo's branch refs/heads/master from Sean Busbey [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=6742627 ] ACCUMULO-2733 adds thrift deserialization to Credentials
          Hide
          ASF subversion and git services added a comment -

          Commit 6742627f20ad58e8e0737baddde952d9d7edba87 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Sean Busbey
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=6742627 ]

          ACCUMULO-2733 adds thrift deserialization to Credentials

          Show
          ASF subversion and git services added a comment - Commit 6742627f20ad58e8e0737baddde952d9d7edba87 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Sean Busbey [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=6742627 ] ACCUMULO-2733 adds thrift deserialization to Credentials
          Hide
          Sean Busbey added a comment -

          result of RB feedback

          Show
          Sean Busbey added a comment - result of RB feedback

            People

            • Assignee:
              Sean Busbey
              Reporter:
              Sean Busbey
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development