You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2020/08/06 08:48:51 UTC
[GitHub] [iceberg] JingsongLi opened a new issue #1304: `select` will overwrite `project` whatever the order in TableScan
JingsongLi opened a new issue #1304:
URL: https://github.com/apache/iceberg/issues/1304
`select` will overwrite `project` whatever the order.
-> table.newScan().select("id").project(SCHEMA.select("data"))
Expected: "data"
Actual: "id"
Should we fix this? Invalid `select` when `project`?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] rdblue closed issue #1304: `select` will overwrite `project` whatever the order in TableScan
Posted by GitBox <gi...@apache.org>.
rdblue closed issue #1304:
URL: https://github.com/apache/iceberg/issues/1304
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] rdblue commented on issue #1304: `select` will overwrite `project` whatever the order in TableScan
Posted by GitBox <gi...@apache.org>.
rdblue commented on issue #1304:
URL: https://github.com/apache/iceberg/issues/1304#issuecomment-670588377
The two methods are alternatives to one another, so I would not expect users to call both of them. I'd be okay with fixing this (open to suggestions on behavior) or simply throwing an exception if both methods are used to configure the same table scan.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] JingsongLi commented on issue #1304: `select` will overwrite `project` whatever the order in TableScan
Posted by GitBox <gi...@apache.org>.
JingsongLi commented on issue #1304:
URL: https://github.com/apache/iceberg/issues/1304#issuecomment-675212776
> The two methods are alternatives to one another, so I would not expect users to call both of them. I'd be okay with fixing this (open to suggestions on behavior) or simply throwing an exception if both methods are used to configure the same table scan.
Thank you for your reply, I think simply throwing an exception is good.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org