[
https://issues.apache.org/jira/browse/TRAFODION-3065?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16469353#comment-16469353
]
ASF GitHub Bot commented on TRAFODION-3065:
-------------------------------------------
Github user zellerh commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1557#discussion_r187144495
--- Diff: core/sql/comexe/ComTdbFastTransport.h ---
@@ -395,9 +399,10 @@ class ComTdbFastExtract : public ComTdb
UInt16 filler_; // 130 - 131
UInt32 childDataRowLen_; // 132 - 135
Int64 modTSforDir_; // 136 - 143
+ UInt16 hdfsIoByteArraySize_; // 144 - 147
--- End diff --
Not that it really matters a lot, but I think this only takes up two bytes,
144-145?
Another comment: This doesn't give a hint at all about what the unit is,
bytes, KB, MB? IMHO, it would be better to just use an int and specify the size
in bytes, to avoid this problem. If you want to keep the KB unit, I would
recommend naming this hdfsIoByteArraySizeInKB_.
This is a short (see comment about unsigned short below). Is 32 MB max IO
byte array size enough?
> Trafodion to support compressed Hive Text formatted tables
> ----------------------------------------------------------
>
> Key: TRAFODION-3065
> URL: https://issues.apache.org/jira/browse/TRAFODION-3065
> Project: Apache Trafodion
> Issue Type: New Feature
> Components: sql-cmp, sql-exe
> Reporter: Selvaganesan Govindarajan
> Assignee: Selvaganesan Govindarajan
> Priority: Major
> Fix For: 2.3
>
>
> Currently, Trafodion doesn't support Hive text formatted compressed files.
> The compressed file support will be provided with the refactored Hdfs scan
> implementation only.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
