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

Include INSERT INTO in Qbeast save contract #5

Closed
osopardo1 opened this issue Aug 16, 2021 · 1 comment · Fixed by #116
Closed

Include INSERT INTO in Qbeast save contract #5

osopardo1 opened this issue Aug 16, 2021 · 1 comment · Fixed by #116
Labels
good first issue Good for newcomers type: enhancement Improvement of existing feature or code

Comments

@osopardo1
Copy link
Member

What went wrong?
Right now on the master, we cannot add new elements to a qbeast table through the INSERT INTO sql command

How to reproduce?

  1. Code that triggered the bug, or steps to reproduce:

On a previously indexed dataframe named df:

df.createOrReplaceTempView("t")
spark.sql("insert into table t (value) values (4)")

And the output is:

org.apache.spark.sql.AnalysisException: QbeastBaseRelation(parquet) does not allow insertion.;
'InsertIntoStatement Relation[value#1297] QbeastBaseRelation(parquet), false, false
+- LocalRelation [col1#1543]

  at org.apache.spark.sql.execution.datasources.PreWriteCheck$.failAnalysis(rules.scala:512)

  1. Branch and commit id:

master ef6b9f45ac54ab156e5b3474e3014b639b2ac827

  1. Spark version:
    3.x

  2. Hadoop version:

2.4.7

  1. Are you running Spark inside a container? Are you launching the app on a remote K8s cluster? Or are you just running the tests in a local computer?
    Local

  2. Stack trace:

@cugni cugni transferred this issue from another repository Sep 23, 2021
@osopardo1 osopardo1 added type: bug Something isn't working type: enhancement Improvement of existing feature or code and removed type: bug Something isn't working labels Sep 28, 2021
@osopardo1 osopardo1 added the good first issue Good for newcomers label Mar 7, 2022
@eavilaes
Copy link
Contributor

eavilaes commented Aug 3, 2022

Solved by #116

@eavilaes eavilaes closed this as completed Aug 3, 2022
@eavilaes eavilaes linked a pull request Aug 3, 2022 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers type: enhancement Improvement of existing feature or code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants