Documentation for PostgreSQL Metadata Store SSL Configuration Settings is incorrect

[navkumar]

Originally filed in druid-io.github.io

On this page in the documentation for the PostgreSQL Metadata Store (http://druid.io/docs/latest/development/extensions-core/postgresql.html), the settings table at the bottom uses incorrect properties that do not work in the latest (0.13.0-incubating) version of Druid.

According to the code here (PostgreSQLMetadataStorageModule.java), the SSL related settings are all prefixed with druid.metadata.postgres.ssl NOT druid.metadata.postgres as indicated on this documentation page. For example, you would enable SSL for PostgreSQL using the something like the following in runtime.properties:

...
# ALL THESE PROPERTIES ARE PREFIXED with "druid.metadata.postgres.ssl"
druid.metadata.postgres.ssl.enableSSL=true
druid.metadata.postgres.ssl.sslMode=verify-full
druid.metadata.postgres.ssl.sslRootCert=<path-to-ca.crt>
...

instead of:

...
# NONE OF THESE ARE READ BY THE POSTGRES METADATA STORAGE EXTENSION
druid.metadata.postgres.enableSSL=true
druid.metadata.postgres.sslMode=verify-full
druid.metadata.postgres.sslRootCert=<path-to-ca.crt>
...

I have updated the options to reflect the updated logic in the code.

[gianm]

LGTM. Will merge even without CI passing; it must be a false positive since this is just a doc change.