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

Cannot migrate query that contains a function over a join column #1372

Open
jasobrown-rs opened this issue Sep 19, 2024 · 0 comments
Open

Cannot migrate query that contains a function over a join column #1372

jasobrown-rs opened this issue Sep 19, 2024 · 0 comments
Labels
bug Something isn't working High priority Created by Linear-GitHub Sync Migrations Created by Linear-GitHub Sync
Milestone

Comments

@jasobrown-rs
Copy link
Contributor

jasobrown-rs commented Sep 19, 2024

Description

When testing readyset with benchbase's tpc-h benchmark, several queries contain a join condition, expressed in the where clause that causes a peculiar error when at least one of the columns is wrapped with a function.

Here is the query

create cache from
          SELECT c_id,
              c_last,
              sum(ol_amount) AS revenue,
              c_city,
              c_phone,
              n_name
              FROM customer,
              oorder,
              order_line,
              nation
              WHERE c_id = o_c_id
              AND c_w_id = o_w_id
              AND c_d_id = o_d_id
              AND ol_w_id = o_w_id
              AND ol_d_id = o_d_id
              AND ol_o_id = o_id
              AND o_entry_d >= '2007-01-02 00:00:00.000000'
              AND o_entry_d <= ol_delivery_d
              AND n_nationkey = ascii(substring(c_state from  1  for  1))
              GROUP BY c_id,
              c_last,
              c_city,
              c_phone,
              n_name
              ORDER BY revenue DESC;

Here is the error, as seen in the mysql client (carriage returns added by me):

ERROR 1105 (HY000): Error during RPC (extend_recipe (in readyset-client/src/controller/rpc.rs:49:26)): 
   SQL SELECT query 'q_6565248d6c10a8b' couldn't be added: MIR node '18' couldn't be lowered to dataflow: 
   Internal error: in readyset-server/src/controller/mir_to_flow.rs:45:5: 
    invariant failed: columns.len() == names.len();

The problem occurs because of this where predicate (which is a join condition):

AND n_nationkey = ascii(substring(c_state from  1  for  1))

Additional data points:

  • I tried several different functions (ascii, length, …) and they all exhibited the same error.
  • I changed the join clause to a naive literal (AND ascii(substring(c_state from 1 for 1)) ='f'), and it successfully migrated.
  • I created a variant where the two functions are over the same table's columns (where ascii(first_name) = ascii(last_name)), and this successfully migrated as well.
  • I moved the join condition, with the function, into the from clause, and it failed to migrate.

Thus, i think it is because at least one of the join condition's columns is wrapped in a function.

Steps to reproduce

create table user (id int, dept_name varchar(32));
create table dept (dept_id int, name varchar(32));
create cache from select user.id, dept.dept_id 
    from user, dept 
    where ascii(user.dept_name) = dept.name;

ReadySet version

git sha b647d97e3f1a0dfbe47168d0814a67f003066bb6

cli command

cargo --locked run --bin readyset -- \
   --upstream-db-url=mysql://root:[email protected]:3306/benchbase --feature-post-lookup
@jasobrown-rs jasobrown-rs added bug Something isn't working High priority Created by Linear-GitHub Sync Migrations Created by Linear-GitHub Sync labels Sep 19, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.43, v.44 Sep 20, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.44, v.45 Sep 30, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.45, v.46 Oct 7, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.46, v.47, v.48 Oct 14, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.48, v.49 Oct 28, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.49, v.50 Nov 4, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.50, v.51, v.52 Nov 11, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.52, v.53, v.54 Nov 25, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.54, v.55, v.56 Dec 9, 2024
@jasobrown-rs jasobrown-rs modified the milestones: v.56, v.57 Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working High priority Created by Linear-GitHub Sync Migrations Created by Linear-GitHub Sync
Projects
None yet
Development

No branches or pull requests

1 participant