[teiid-issues] [JBoss JIRA] (TEIID-4883) Teiid Excel Translator should filter blank cells from header row
Ramesh Reddy (JIRA)
issues at jboss.org
Tue May 2 14:39:00 EDT 2017
[ https://issues.jboss.org/browse/TEIID-4883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13400924#comment-13400924 ]
Ramesh Reddy commented on TEIID-4883:
-------------------------------------
Yes, instead of {{getAllowEmptyCells}}, I propose we say {{getIgnoreEmptyCells}} when {{true}}, all the 3 columns are returned ignoring/skipping the empty space, and if {{false}} only first 2 columns are returned.
> Teiid Excel Translator should filter blank cells from header row
> ----------------------------------------------------------------
>
> Key: TEIID-4883
> URL: https://issues.jboss.org/browse/TEIID-4883
> Project: Teiid
> Issue Type: Enhancement
> Components: Misc. Connectors
> Reporter: David Vilaverde
> Assignee: Steven Hawkins
> Priority: Minor
> Fix For: Open To Community
>
> Attachments: fr-demo1-sample-data.xls
>
>
> The teiid translator for excel will throw a duplicate column name exception when using a spreadsheet that has 2 or more columns where the header row doesn't specify a header name. In the attached spreadsheet it's difficult to see that rows E-H should be deleted from the dataset.
> The proposed enhancement is to allow the translator for excel to exclude any trailing columns where there is no column name defined.
> I took a shot at making this enhancement and created a PR at https://github.com/teiid/teiid/pull/951
> If this PR is not going to be accepted, can the methods in the ExcelMetadataProcessor me changed from private to protected so that the processor can be extended easily?
--
This message was sent by Atlassian JIRA
(v7.2.3#72005)
More information about the teiid-issues
mailing list