STORM-2202 [Storm SQL] Document how to use supported connectors and formats #1777
Conversation
| #### CSV | ||
|
|
||
| It uses `Standard RFC4180 CSV Parser` and doesn't need any other properties. |
There was a problem hiding this comment.
Minor. How about add a link to RFC4180? It is convenient for users who want to look.
There was a problem hiding this comment.
Yes that would be a good idea. Will address.
| | Redis | org.apache.storm:storm-sql-redis | `redis://:[password]@host:port/[dbIdx]` | No | Yes | Yes | ||
| | MongoDB | org.apache.stormg:storm-sql-mongodb | `mongodb://[username:password@]host1[:port1][,host2[:port2],...[,hostN[:portN]]][/[database][?options]]` | No | Yes | Yes | ||
|
|
There was a problem hiding this comment.
Do you mind this PR including STORM-2082(storm-sql-hdfs, after #1778 getting merged)? I hope storm 1.1.0 include these changes.
There was a problem hiding this comment.
Sure. I'll update when STORM-2082 is merged to master. I'd like to ask you a favor of providing the content for storm-sql-hdfs from #1778 so that I can easily take it.
|
Thanks @HeartSaVioR Just two minor comments. Others looks good to me. +1 |
HeartSaVioR
force-pushed
the
STORM-2202
branch
from
102bb13
to
1b3848d
Compare
HeartSaVioR
force-pushed
the
STORM-2202
branch
from
1b3848d
to
957fcab
Compare
|
I'll merge this now since it doesn't need binding +1 (documentation) and @vesense confirms it's OK. |
Copy setting up external data sources to reference page, and add description on data sources (connectors) and formats.
@vesense Since you authored many of them, I'd be happy if you can take a look.