Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-1443

Add authentication support for Cassandra adapter

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.11.0
    • Component/s: cassandra
    • Labels:
      None
    • Environment:

      Test on Apache Cassandra 3.9 and Calcite Cassandra adapter

    • Flags:
      Patch

      Description

      Calcite Cassandra adapter does not support Cassandra authenticator option is set to PasswordAuthenticator case.

      Cassandra adapter operand now only support host and keyspace option, I think need add username and password option.

      1. cass_auth.patch
        2 kB
        Danilo Chang

        Activity

        Hide
        ray2501 Danilo Chang added a comment -

        In the attachment, I add authentication support for Cassandra adapter. However, I need modify CassandraSchema class interface and operand option need add username and password (so it is an INCOMPATIBILITY change). I don't konw it is OK or not.

        So I create this item and hope that have better solution for authentication support for Cassandra adapter.

        Show
        ray2501 Danilo Chang added a comment - In the attachment, I add authentication support for Cassandra adapter. However, I need modify CassandraSchema class interface and operand option need add username and password (so it is an INCOMPATIBILITY change). I don't konw it is OK or not. So I create this item and hope that have better solution for authentication support for Cassandra adapter.
        Hide
        michaelmior Michael Mior added a comment -

        Thanks for the patch! I'd prefer to keep backwards compatibility, but check out this branch and see if it works for you. If it does I'll merge this soon and it will be in the next release of Calcite.

        Show
        michaelmior Michael Mior added a comment - Thanks for the patch! I'd prefer to keep backwards compatibility, but check out this branch and see if it works for you. If it does I'll merge this soon and it will be in the next release of Calcite.
        Hide
        ray2501 Danilo Chang added a comment -

        It works for me. Thank you for your help.

        Show
        ray2501 Danilo Chang added a comment - It works for me. Thank you for your help.
        Hide
        julianhyde Julian Hyde added a comment -
        Show
        julianhyde Julian Hyde added a comment - Does https://calcite.apache.org/docs/cassandra_adapter.html need an update?
        Hide
        michaelmior Michael Mior added a comment -

        I can update to add information on authentication if you prefer, but the current docs are still valid.

        Show
        michaelmior Michael Mior added a comment - I can update to add information on authentication if you prefer, but the current docs are still valid.
        Hide
        julianhyde Julian Hyde added a comment -

        Yes, the current doc is valid. But someone would have to read the code to find out about the "username" and "password" properties, right? It would be better if all available properties are described in the doc.

        Show
        julianhyde Julian Hyde added a comment - Yes, the current doc is valid. But someone would have to read the code to find out about the "username" and "password" properties, right? It would be better if all available properties are described in the doc.
        Hide
        michaelmior Michael Mior added a comment -

        I updated to add a line explaining that the username and password can be specified.

        Show
        michaelmior Michael Mior added a comment - I updated to add a line explaining that the username and password can be specified.
        Show
        michaelmior Michael Mior added a comment - Fixed in https://git-wip-us.apache.org/repos/asf?p=calcite.git;a=commit;h=d0f7dd3
        Hide
        julianhyde Julian Hyde added a comment -

        Resolved in release 1.11.0 (2017-01-11).

        Show
        julianhyde Julian Hyde added a comment - Resolved in release 1.11.0 (2017-01-11).

          People

          • Assignee:
            michaelmior Michael Mior
            Reporter:
            ray2501 Danilo Chang
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development