Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-478

Correct example/csv/TUTORIAL.md according to the current package names

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0-incubating
    • Fix Version/s: 1.0.0-incubating
    • Component/s: None
    • Labels:

      Description

      example/csv/TUTORIAL.md references non-existing classes after package renaming.

      The tutorial needs to be corrected and Optiq references should be chaged to Calcite (e.g. optiq-csv -> calcite-csv)

      1. CALCITE-478-1.patch
        55 kB
        Sivaramakrishnan Narayanan

        Activity

        Hide
        julianhyde Julian Hyde added a comment -

        example/csv/TUTORIAL.md and example/csv/README.md only live under example/csv for historical reasons. TUTORIAL.md should be moved under incubator-calcite/doc, and example/csv/README.md should be removed (moving any useful content elsewhere).

        Show
        julianhyde Julian Hyde added a comment - example/csv/TUTORIAL.md and example/csv/README.md only live under example/csv for historical reasons. TUTORIAL.md should be moved under incubator-calcite/doc, and example/csv/README.md should be removed (moving any useful content elsewhere).
        Hide
        snarayanan Sivaramakrishnan Narayanan added a comment - - edited

        Attaching patch with doc changes. Removed README.md and moved TUTORIAL.md to doc directory.

        Also changed optiq-csv to calcite-example-csv (which is the project name in the pom file).

        Show
        snarayanan Sivaramakrishnan Narayanan added a comment - - edited Attaching patch with doc changes. Removed README.md and moved TUTORIAL.md to doc directory. Also changed optiq-csv to calcite-example-csv (which is the project name in the pom file).
        Show
        snarayanan Sivaramakrishnan Narayanan added a comment - Created PR in github: https://github.com/apache/incubator-calcite/pull/21
        Show
        julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/5010407 and http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/a640e58 .

          People

          • Assignee:
            julianhyde Julian Hyde
            Reporter:
            vladimirsitnikov Vladimir Sitnikov
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development