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)