[superset] branch amitmiran137-patch-5 created (now 7afe706)

[am...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

amitmiran pushed a change to branch amitmiran137-patch-5
in repository https://gitbox.apache.org/repos/asf/superset.git.


      at 7afe706  fix: preventing sql lab None limit value

This branch includes the following new commits:

     new 7afe706  fix: preventing sql lab None limit value

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[superset] 01/01: fix: preventing sql lab None limit value

[am...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

amitmiran pushed a commit to branch amitmiran137-patch-5
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 7afe706f42fa2b477871f4ff2b7a0058884ecb97
Author: Amit Miran <47...@users.noreply.github.com>
AuthorDate: Tue Oct 19 20:42:12 2021 +0300

    fix: preventing sql lab None limit value
---
 superset/sql_lab.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/superset/sql_lab.py b/superset/sql_lab.py
index f5c9e42..957bde7 100644
--- a/superset/sql_lab.py
+++ b/superset/sql_lab.py
@@ -221,7 +221,7 @@ def execute_sql_statement(  # pylint: disable=too-many-arguments,too-many-locals
     ):
         if SQL_MAX_ROW and (not query.limit or query.limit > SQL_MAX_ROW):
             query.limit = SQL_MAX_ROW
-        if query.limit:
+        if query.limit and increased_limit:
             # We are fetching one more than the requested limit in order
             # to test whether there are more rows than the limit.
             # Later, the extra row will be dropped before sending