zhztheplayer commented on PR #8839:
URL:
https://github.com/apache/incubator-gluten/pull/8839#issuecomment-2705604506
> > > After try to run this POC, got error `Caused by:
io.github.zhztheplayer.velox4j.exception.VeloxException: Exception:
VeloxUserError Error Source: USER Error Code: INVALID_ARGUMENT Reason: Splits
can be associated only with leaf plan nodes which require splits. Plan node ID
5 doesn't refer to such plan node.`
> >
> >
> > You can try the latest code.
>
> Thanks for reply. I rerun with last commit and gotcha
>
> ```java
> Caused by: io.github.zhztheplayer.velox4j.exception.VeloxException:
com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized
field "sources" (class io.github.zhztheplayer.velox4j.plan.TableScanNode), not
marked as ignorable (4 known properties: "id", "tableHandle", "outputType",
"assignments"])
> at [Source: UNKNOWN; byte offset: #UNKNOWN] (through reference chain:
io.github.zhztheplayer.velox4j.plan.TableScanNode["sources"])
> ```
>
> I looked at TableScanNode and notice that json getter exist on "sources"
but this field is absence. I try rebuild velox4j with Serde change (add disable
for jackson DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES) and got
>
> ```java
> Caused by: io.github.zhztheplayer.velox4j.exception.VeloxException:
couldn't find key name in dynamic object
> at io.github.zhztheplayer.velox4j.jni.JniWrapper.executeQuery(Native
Method)
> at
io.github.zhztheplayer.velox4j.jni.JniApi.executeQuery(JniApi.java:52)
> at io.github.zhztheplayer.velox4j.query.Queries.execute(Queries.java:14)
> at
org.apache.gluten.table.runtime.operators.GlutenCalOperator.processElement(GlutenCalOperator.java:97)
> ```
>
> SQL query
>
> ```sql
> CREATE TABLE srcTbl (id INT, price INT, name STRING) WITH
('connector'='datagen');
> CREATE TABLE snkTbl (id INT, price INT) WITH ('connector'='blackhole');
> INSERT INTO snkTbl SELECT id, price FROM srcTbl WHERE price > 10;
> ```
The error is likely because this velox4j change
https://github.com/velox4j/velox4j/commit/b1e66a260b48c792eb067c5c70307d932f12b30f
is not included in the release yet. I will release a newer version of velox4j
then let you know.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
