MaxGekk commented on issue #26102: [SPARK-29448][SQL] Support the `INTERVAL`
type by Parquet datasource
URL: https://github.com/apache/spark/pull/26102#issuecomment-574044689
@cloud-fan Should I close this PR?
This is an
MaxGekk commented on issue #26102: [SPARK-29448][SQL] Support the `INTERVAL`
type by Parquet datasource
URL: https://github.com/apache/spark/pull/26102#issuecomment-549154070
> So there should no longer be a difference in reading/writing "1 day" as an
interval after the previous change?
MaxGekk commented on issue #26102: [SPARK-29448][SQL] Support the `INTERVAL`
type by Parquet datasource
URL: https://github.com/apache/spark/pull/26102#issuecomment-54970
> let's wait for #26134 and revisit this later.
@cloud-fan @srowen Could you take a look at the PR one more
MaxGekk commented on issue #26102: [SPARK-29448][SQL] Support the `INTERVAL`
type by Parquet datasource
URL: https://github.com/apache/spark/pull/26102#issuecomment-542374895
In general, are you ok with the proposed changes?
MaxGekk commented on issue #26102: [SPARK-29448][SQL] Support the `INTERVAL`
type by Parquet datasource
URL: https://github.com/apache/spark/pull/26102#issuecomment-541462382
TODO: Need to exclude the check
MaxGekk commented on issue #26102: [SPARK-29448][SQL] Support the `INTERVAL`
type by Parquet datasource
URL: https://github.com/apache/spark/pull/26102#issuecomment-541462167
> My only concern is loss of precision, but what can you do, I suppose,
besides not support it?
Just in
MaxGekk commented on issue #26102: [SPARK-29448][SQL] Support the `INTERVAL`
type by Parquet datasource
URL: https://github.com/apache/spark/pull/26102#issuecomment-541449873
@cloud-fan @HyukjinKwon @dongjoon-hyun @srowen Could you take a look at the
PR, please.