[STORM-2845] Drop standalone mode of Storm SQL

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Component/s: storm-sql
• Labels: pull-request-available
• Assignee: Jungtaek Lim
• Reporter: Jungtaek Lim
• Created: 06/Dec/17 07:13
• Updated: 15/Dec/17 09:56
• Resolved: 15/Dec/17 09:56

Description

Quoting discussion again:

We have been exposing "standalone mode" of Storm SQL which leverages Storm SQL in a JVM process rather than composing topology and run.
At a start we implemented both standalone and trident modes with same approach, but while we improved Storm SQL by leveraging more features on Calcite, we addressed only trident mode, and now twos are diverged.

I guess there is likely no actual user on standalone mode since its classes are exposed but we didn't document it. I know a case, but the source codes on standalone mode code are migrated to the project (and modified to conform to the project) and the project no longer depends on Storm SQL.

If we all don't have any other case, how about dropping it and only concentrate to trident mode?
(Btw, I'm trying to replace the backend on Storm SQL from Trident to Streams API, which may make the mode name obsolete, but after dropping standalone mode we don't even need the name for mode since there will be only one mode.)

Discussion link:
https://mail-archives.apache.org/mod_mbox/storm-dev/201712.mbox/%3CCAF5108g3yEQPWO-UPQaJmoqkN4%2BoZgnH64pvKs2ARju4ySUB4Q%40mail.gmail.com%3E

Issue Links

• links to: GitHub Pull Request #2447