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. Text File
            cass_auth.patch
            2 kB
            Danilo Chang

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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
            Permalink
            julianhyde Julian Hyde added a comment -

            Does https://calcite.apache.org/docs/cassandra_adapter.html need an update?

            Show
            julianhyde Julian Hyde added a comment - Does https://calcite.apache.org/docs/cassandra_adapter.html need an update?
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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.
            Hide
            Permalink
            michaelmior Michael Mior added a comment -

            Fixed in https://git-wip-us.apache.org/repos/asf?p=calcite.git;a=commit;h=d0f7dd3

            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
            Permalink
            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