[BEAM-2798] rename simpleQuery to query and query to queryMulti

• Type: Sub-task
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: Not applicable
• Component/s: dsl-sql
• Labels: Done dsl_sql_merge dsl_sql_review
• Assignee: Mingmin Xu
• Reporter: Mingmin Xu
• Created: 24/Aug/17 00:25
• Updated: 24/Jul/20 20:11
• Resolved: 24/Aug/17 17:18

Description

One other question from playing around with things more: simpleQuery vs query method names. Would it be better if they were named query and queryMulti, respectively? Two potential reasons why that might be better: 1) make simple things simple, e.g., the single input version should be simpler to express. 2) method names typically should be actions/verbs, but simpleQuery makes it sound like a noun/object.

Issue Links

• links to: GitHub Pull Request #3752