docs(databases): Update pinot.mdx to incorporate username and password based connection.

[raamri]

SUMMARY

Updating the documentation to incorporate the username and password based sql for connecting to Apache Pinot.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

N/A.

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[github-actions]

Congrats on making your first PR and thank you for contributing to Superset! 🎉 ❤️

We hope to see you in our Slack community too! Not signed up? Use our Slack App to self-register.

[sfirke]

Hi and thanks for this! I thought "I know someone shared this in Slack, I'll compare them" -- and then I realized I think that was you 😁

In your Slack config it began simply pinot://, here you have pinot+http -- is that +http needed? And in Slack you ended it &verify_ssl=true instead of a trailing slash.

[sfirke]

Looks like linting check failed due to:

Error: Please format your PR title to match: ^(build|chore|ci|docs|feat|fix|perf|refactor|style|test|other)(\(.+\))?(\!)?:\s.+!

Can you change the PR title to begin docs(databases): ?

[raamri]

Hi @sfirke sfirke! Updated the PR title and the doc based on your suggestions.

1. verify_ssl=true is actually needed.
2. +http is not needed. Adding that would also do no harm though. Still, removing it.
   Thanks!

[sfirke]

Looks good to me!

[sfirke]

thanks @raamri !