James Taylor

Results 33 comments of James Taylor

Ping @apurtell - have you had a chance to manually test this yet? Just wanted to see if you're blocked or not for someone reviewing this.

Please squash all commits into a single commit.

Thanks for the PR, @zeeshanasghar. Please file a JIRA and prepend the JIRA number (i.e. PHOENIX-####) to your git commit message. Would you mind reviewing, @samarthjain ?

@zeeshanasghar - please amend your commit message to "PHOENIX-2627 Utility classes should not have public constructors" so that conversations on the pull request are reflected in the JIRA.

FYI, @shubhamtagra - we continue to run into this issue. It occurs when the cluster is overloaded. Essentially invalid files are loaded into the cache then every time a read...

Verified that the issue occurs with both read-through and async modes. Unfortunately I had to disable caching due to this issue.

``` io.prestosql.spi.PrestoException: Error opening Hive split s3://lyftqubole-iad/qubole/t/yrotsi/PRODUCTION/stage/subscriptions_history_7cdfa8/ds=2020-12-01/20201202_140043_07242_5a5n7_adae9373-a180-4d2e-9ae8-57f3f71f7e44 (offset=0, length=67571396): can not read class org.apache.parquet.format.FileMetaData: don't know what type: 15 at io.prestosql.plugin.hive.parquet.ParquetPageSourceFactory.createPageSource(ParquetPageSourceFactory.java:242) at io.prestosql.plugin.hive.parquet.ParquetPageSourceFactory.createPageSource(ParquetPageSourceFactory.java:132) at io.prestosql.plugin.hive.HivePageSourceProvider.createHivePageSource(HivePageSourceProvider.java:178) at io.prestosql.plugin.hive.HivePageSourceProvider.createPageSource(HivePageSourceProvider.java:105) at io.prestosql.plugin.base.classloader.ClassLoaderSafeConnectorPageSourceProvider.createPageSource(ClassLoaderSafeConnectorPageSourceProvider.java:57) at...

The correct sizing of the pool is really related to the number of worker nodes. Sizing too small causes many more queries to timeout while sizing too large can cause...

@stagraqubole - I can take this one if you'd like.

@findepi - would it be possible to elaborate on what "more pushdown" means? It's not clear which TODOs you mean. Ideally, if we could break this down into specific pushdown...