Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sample operator takes effect after the execution of the query, expected before #6340

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Labels
bug Something isn't working normal SQL

Comments

@monetdb-team
Copy link

Date: 2017-07-05 11:56:48 +0200
From: pavlos <<pavlos.katsogridakis>>
To: SQL devs <>
Version: 11.23.3 (Jun2016)
CC: @njnes, @yzchang

Last updated: 2017-08-02 10:21:31 +0200

Comment 25409

Date: 2017-07-05 11:56:48 +0200
From: pavlos <<pavlos.katsogridakis>>

User-Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:54.0) Gecko/20100101 Firefox/54.0
Build Identifier:

Tried to run the tpch 03 query, taking a 10% sample on each table,
so I appended sample 0.1; to the query.
I expected the query to take a 10% sample of the tables and then run the
query. Instead, it executed the query and then it took a 10% sample.

Reproducible: Always

Steps to Reproduce:

1.Setup tpch with sf10
2.Append sample 0.1; to the 03.sql query

Comment 25413

Date: 2017-07-05 18:33:37 +0200
From: @njnes

indeed the sample operator (like limit) does it job on top of the result. Sampling per input should be done by hand.

Comment 25425

Date: 2017-07-06 11:14:36 +0200
From: @yzchang

Yeah, we got that sampling the base tables requires some manual work.

But how about the combined use of SAMPLE and LIMIT? Shouldn't SAMPLE be executed before LIMIT? Or should we actually disallow their combined use, since they both reduce the result set, only using different algorithms?

Comment 25539

Date: 2017-08-02 10:21:31 +0200
From: @sjoerdmullender

Fixed in Jul2017-SP1 release.

@monetdb-team monetdb-team added bug Something isn't working normal SQL labels Nov 30, 2020
@sjoerdmullender sjoerdmullender added this to the Ancient Release milestone Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working normal SQL
Projects
None yet
Development

No branches or pull requests

2 participants