From 8a253825946e1e083673ad68e5352a95050291a9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timo=20Westk=C3=A4mper?= <timo.westkamper@gmail.com>
Date: Wed, 25 Nov 2015 20:28:37 +0200
Subject: [PATCH] Deprecate getResults with arguments

---
 .../java/com/querydsl/sql/AbstractSQLQuery.java | 17 +++++++++++++++--
 .../com/querydsl/sql/QueryPerformanceTest.java  |  4 ++--
 .../test/java/com/querydsl/sql/SelectBase.java  |  2 +-
 3 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/querydsl-sql/src/main/java/com/querydsl/sql/AbstractSQLQuery.java b/querydsl-sql/src/main/java/com/querydsl/sql/AbstractSQLQuery.java
index 21f12ccfef..677a1665a5 100644
--- a/querydsl-sql/src/main/java/com/querydsl/sql/AbstractSQLQuery.java
+++ b/querydsl-sql/src/main/java/com/querydsl/sql/AbstractSQLQuery.java
@@ -247,14 +247,27 @@ protected void endContext(SQLListenerContext context) {
     }
 
     /**
-     * Get the results as an JDBC result set
+     * Get the results as an JDBC ResultSet
      *
      * @param exprs the expression arguments to retrieve
      * @return results as ResultSet
+     * @deprecated Use @{code select(..)} to define the projection and {@code getResults()} to obtain
+     *             the result set
      */
+    @Deprecated
     public ResultSet getResults(Expression<?>... exprs) {
-        queryMixin.setProjection(exprs);
+        if (exprs.length > 0) {
+            queryMixin.setProjection(exprs);
+        }
+        return getResults();
+    }
 
+    /**
+     * Get the results as an JDBC ResultSet
+     *
+     * @return results as ResultSet
+     */
+    public ResultSet getResults() {
         SQLListenerContextImpl context = startContext(connection(), queryMixin.getMetadata());
         String queryString = null;
         List<Object> constants = ImmutableList.of();
diff --git a/querydsl-sql/src/test/java/com/querydsl/sql/QueryPerformanceTest.java b/querydsl-sql/src/test/java/com/querydsl/sql/QueryPerformanceTest.java
index 5036325a84..1de2945921 100644
--- a/querydsl-sql/src/test/java/com/querydsl/sql/QueryPerformanceTest.java
+++ b/querydsl-sql/src/test/java/com/querydsl/sql/QueryPerformanceTest.java
@@ -158,8 +158,8 @@ public void run(int times) throws Exception {
                 for (int i = 0; i < times; i++) {
                     QCompanies companies = QCompanies.companies;
                     SQLQuery<?> query = new SQLQuery<Void>(conn, conf);
-                    ResultSet rs = query.from(companies)
-                            .where(companies.id.eq((long) i)).getResults(companies.name);
+                    ResultSet rs = query.select(companies.name).from(companies)
+                            .where(companies.id.eq((long) i)).getResults();
                     try {
                         while (rs.next()) {
                             rs.getString(1);
diff --git a/querydsl-sql/src/test/java/com/querydsl/sql/SelectBase.java b/querydsl-sql/src/test/java/com/querydsl/sql/SelectBase.java
index 5de800c107..a5fba82e79 100644
--- a/querydsl-sql/src/test/java/com/querydsl/sql/SelectBase.java
+++ b/querydsl-sql/src/test/java/com/querydsl/sql/SelectBase.java
@@ -756,7 +756,7 @@ public void Full_Join() throws SQLException {
 
     @Test
     public void GetResultSet() throws IOException, SQLException {
-        ResultSet results = query().from(survey).getResults(survey.id, survey.name);
+        ResultSet results = query().select(survey.id, survey.name).from(survey).getResults();
         while (results.next()) {
             assertNotNull(results.getObject(1));
             assertNotNull(results.getObject(2));