[
https://issues.apache.org/jira/browse/TAJO-1730?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14746942#comment-14746942
]
ASF GitHub Bot commented on TAJO-1730:
--------------------------------------
Github user jihoonson commented on a diff in the pull request:
https://github.com/apache/tajo/pull/719#discussion_r39596419
--- Diff: tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java
---
@@ -1398,24 +1398,18 @@ public ScanNode visitRelation(PlanContext context,
Stack<Expr> stack, Relation e
return targets;
}
- private void updatePhysicalInfo(PlanContext planContext, TableDesc desc)
{
- if (desc.getUri() != null &&
- !desc.getMeta().getStoreType().equals("SYSTEM") &&
- !desc.getMeta().getStoreType().equals("FAKEFILE") && // FAKEFILE
is used for test
- PlannerUtil.isFileStorageType(desc.getMeta().getStoreType())) {
+ private void updatePhysicalInfo(TableDesc desc) {
+
+ // FAKEFILE is used for test
+ if (!desc.getMeta().getStoreType().equals("SYSTEM") &&
!desc.getMeta().getStoreType().equals("FAKEFILE")) {
try {
- Path path = new Path(desc.getUri());
- FileSystem fs =
path.getFileSystem(planContext.queryContext.getConf());
- FileStatus status = fs.getFileStatus(path);
- if (desc.getStats() != null && (status.isDirectory() ||
status.isFile())) {
- ContentSummary summary = fs.getContentSummary(path);
- if (summary != null) {
- long volume = summary.getLength();
- desc.getStats().setNumBytes(volume);
- }
+ if (desc.getStats() != null) {
+
desc.getStats().setNumBytes(storage.getTableVolumn(desc.getUri()));
}
} catch (Throwable t) {
--- End diff --
This line seems to be changed to catch UnsupportedException.
> JDBC Tablespace support
> -----------------------
>
> Key: TAJO-1730
> URL: https://issues.apache.org/jira/browse/TAJO-1730
> Project: Tajo
> Issue Type: Improvement
> Components: Storage
> Reporter: Hyunsik Choi
> Assignee: Hyunsik Choi
> Fix For: 0.11.0
>
> Attachments: TAJO-1730.patch, TAJO-1730_2.patch, TAJO-1730_3.patch
>
>
> The main objective of this issue is to implement JDBC-based storage and its
> Tablespace implementation.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
