Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1418#discussion_r149979447
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/impl/CarbonTableReader.java
---
@@ -271,14 +271,20 @@ private CarbonTable
parseCarbonMetadata(SchemaTableName table) {
// If table is not previously cached, then:
// Step 1: get store path of the table and cache it.
- String storePath = config.getStorePath();
// create table identifier. the table id is randomly generated.
cache.carbonTableIdentifier =
new CarbonTableIdentifier(table.getSchemaName(),
table.getTableName(),
UUID.randomUUID().toString());
+ String storePath = config.getStorePath();
--- End diff --
Add TODO saying that developer should pass table path not store path
---
