[ https://jira.jboss.org/browse/TEIID-1064?page=com.atlassian.jira.plugin.s... ] Steve Hawkins updated TEIID-1064: --------------------------------- Fix Version/s: 8.0 -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/TEIID-1064?page=com.atlassian.jira.plugin... ] Filip Nguyen commented on TEIID-1064: ------------------------------------- There are basically 2 ways to implement this. Either using older GData java-library [1] (note this library is being migrated to newer API [4], migration not yet done) that implements Google Spreadsheet API [2] or using Google Visualization Data Protocol [3]. We can offer the best user experience with implementing [3] because many constructs can be pushed down with it as it supports sort of a query language [5]. For broader Google services integration, the best option in many cases is to implement new API [4]. We believe this is not true in the case of Spreadsheet service.New Spreadsheet API probably still won't feature Query language. [1] http://code.google.com/p/gdata-java-client/ [2] https://developers.google.com/google-apps/spreadsheets/ [3] https://developers.google.com/chart/interactive/docs/dev/implementing_dat... [4] http://code.google.com/p/google-api-java-client/ [5] https://developers.google.com/chart/interactive/docs/querylanguage -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/TEIID-1064?page=com.atlassian.jira.plugin... ] Filip Nguyen commented on TEIID-1064: ------------------------------------- Created pull request. Commit dcb8e01 in https://github.com/nguyenfilip/teiid-1 -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/TEIID-1064?page=com.atlassian.jira.plugin... ] Ramesh Reddy updated TEIID-1064: -------------------------------- Fix Version/s: 8.2 (was: Open To Community) Great work Filip, any documents? See this page for the other translators https://docs.jboss.org/author/display/TEIID/Built-in+Translators We will get it merged into master before 8.2 CR1. thanks! -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/TEIID-1064?page=com.atlassian.jira.plugin... ] Steven Hawkins commented on TEIID-1064: --------------------------------------- This looks very good. I've been doing a review and making some modifications along the way. I have some proposed changes that we can first push to you or just get them into master and then make additional refinements. - remove the getAsCsv procedure and just rely on the built-in textAgg against the appropriate source query selecting the columns, limit, and offset. is there a reason to not use textAgg? - add the translator/ra to the embedded kit - use json instead of csv for results. there is additional metadata available and the csv library can be removed. - remove the use of the datatype metadata overrides. the use of jaxb has been discouraged and with json metadata we have a better guess at the types. Other considerations: - allow the integration tests to be run in a specific profile (but they would probably need their own harness to setup the expected data) - use JSONSimple for json processing to cut down on embedded kit size - the GDataClientLoginAPI retry delay seems large, should it be configurable? also won't retry only be needed on ServiceExceptions, shouldn't an IOException be immediately rethrown? -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/TEIID-1064?page=com.atlassian.jira.plugin... ] Steven Hawkins resolved TEIID-1064. ----------------------------------- Resolution: Done Decided to leave modules as is with the idea that google support will be expanded within the existing set rather than adding new. The ra is mostly generic - and the spreadsheet batch size ra property may not be needed in the future. The admin and reference docs have been updated and a release note entry added. A quick start will be added also. Filip, if you have time, can you submit a blog posting for this work as well? -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/TEIID-1064?page=com.atlassian.jira.plugin... ] Steven Hawkins closed TEIID-1064. --------------------------------- -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira