You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@calcite.apache.org by "Marc Prud'hommeaux (JIRA)" <ji...@apache.org> on 2014/07/08 05:38:33 UTC
[jira] [Commented] (OPTIQ-296) Re-organize package structure
[ https://issues.apache.org/jira/browse/OPTIQ-296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14054479#comment-14054479 ]
Marc Prud'hommeaux commented on OPTIQ-296:
------------------------------------------
I took a stab just doing the simple renames in the spreadsheet, and got everything building and all the tests passing. I didn't do any of the other proposed refactoring aside from package renames. Also, I had to disable the import ordering checkstyle rule since there were zillions of failures; if the PRs are accepted, I can go through and get the imports ordered correctly.
The pull requests are at:
https://github.com/apache/incubator-optiq-linq4j/pull/1
https://github.com/apache/incubator-optiq/pull/4
https://github.com/apache/incubator-optiq-csv/pull/1
If you check them out to test, note that you need to do mvn install on each module in order for the subsequent modules to build.
> Re-organize package structure
> -----------------------------
>
> Key: OPTIQ-296
> URL: https://issues.apache.org/jira/browse/OPTIQ-296
> Project: Optiq
> Issue Type: Bug
> Reporter: GitHub Import
> Labels: github-import
>
> As we move code to the `org.apache.optiq` namespace, there is an opportunity to re-organize the package structure of the code.
> Proposed new structure is <a href="https://docs.google.com/spreadsheets/d/1SdLgVh0Zxdk-Hz4mVC7lgzNFFMI5InXu2lsqCh7upeo/edit?usp=sharing">in a spreadsheet</a>.
> ---------------- Imported from GitHub ----------------
> Url: https://github.com/julianhyde/optiq/issues/296
> Created by: [julianhyde|https://github.com/julianhyde]
> Labels:
> Created at: Thu May 29 21:09:56 CEST 2014
> State: open
--
This message was sent by Atlassian JIRA
(v6.2#6252)