Bug 1456401

Summary: PrestoDB - upstream bugs which needs to be included
Product: [JBoss] JBoss Data Virtualization 6 Reporter: Juraj Duráni <jdurani>
Component: Teiid, DocumentationAssignee: David Le Sage <dlesage>
Status: CLOSED CURRENTRELEASE QA Contact: Jan Stastny <jstastny>
Severity: high Docs Contact:
Priority: high    
Version: 6.4.0CC: blafond, drieden, gjospin, jschatte, jstastny, thauser, vhalbert
Target Milestone: DR1   
Target Release: 6.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-02-13 17:34:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Juraj Duráni 2017-05-29 10:18:44 UTC
Description of problem:
Here is the list of PrestoDB translator bugs which should be included in JDV 6.4.0.


TEIID-4676 PrestoDB translator - convert/cast is not properly translated/pushed
TEIID-4677 PrestoDB translator - date/time/timestamp values are pushed in Teiid's format ({d/t/ts '...'})
TEIID-4678 PrestoDB translator - log function not translated correctly
TEIID-4682 PrestoDB translator - cannot use temp tables - PrestoDB could not commit transaction
TEIID-4683 PrestoDB translator - dayof* functions not translated correctly
TEIID-4684 PrestoDB translator - format* functions - invalid conversion to 'timestamp with timezone'
TEIID-4686 PrestoDB translator - ordering of joined tables fails
TEIID-4687 PrestoDB translator - unsupported corelated subquery
TEIID-4694 PrestoDB translator - NULL values not supported in SemiJoin
TEIID-4706 PrestoDBMetadataProcessor load metadata failed due to Duplicate Table

Comment 1 Juraj Duráni 2017-05-29 10:23:02 UTC
Adding TEIID-4685

Comment 2 Juraj Duráni 2017-05-29 10:23:48 UTC
Adding TEIID-4679

Comment 3 Juraj Duráni 2017-05-29 10:24:53 UTC
Adding TEIID-4680 and TEIID-4681

Comment 7 jolee 2017-05-30 18:32:07 UTC
If it's for 6.4 and not as part of an update release... I'm not concerned.

Comment 11 Juraj Duráni 2017-06-07 04:51:09 UTC
Van,

I do not think this should be targeted for DR1. Moreover, none of the fixes has been in any 6.4.0 build, as I have reported this based on testing with 6.4.0 builds. Failing QA as this is not yet fixed.

Comment 13 Jan Stastny 2018-01-08 15:55:55 UTC
Following JIRA issues were resolved by change of project documentation, we need to incorporate them in product docs as well. Assigning to David.

TEIID-4686
TEIID-4694
TEIID-4706

See the Tips and Notes in [1]

https://teiid.gitbooks.io/documents/content/reference/PrestoDB_Translator.html

Comment 14 David Le Sage 2018-01-08 23:57:32 UTC
I have added all of these into the PrestoDB section.

Comment 15 Jan Stastny 2018-01-09 11:05:51 UTC
There are issues in wording of the new content:

1.
>The translator property supports OrderBy
supports OrderBy -> supportsOrderBy

2.
> To check whether support your version supports null as valid values in a subquery see if you can produce a related error.
Remove the sentence, it is unwieldy and adds no information value.

3.
> PrestoDB does not support transaction. To work around this, define a no-tx-datasource.
Reword similar to:
PrestoDB does not support transactions. To overcome issues caused by this limitation, define the datasource as non-transactional.

Comment 16 David Le Sage 2018-01-09 23:38:14 UTC
1. Corrected and formatted.
2. Removed.
3. Replaced.

Comment 17 Jan Stastny 2018-01-10 12:56:13 UTC
1.
> The translator property supportsOrderBy which can be used to disable Order by in some specific situations. 
remove which

Comment 18 David Le Sage 2018-01-12 04:45:09 UTC
Wording amended.