Skip to content

Commit

Permalink
Merge pull request #18 from Matts966/alphasql
Browse files Browse the repository at this point in the history
Remove bin/ prefixes because binaries generated automatically and remotely
  • Loading branch information
Matts966 authored Jul 21, 2020
2 parents 976b452 + b91c2cf commit d7465c8
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 15 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
mkdir alphasql_darwin_amd64
mv ./bazel-bin/alphasql/dag ./alphasql_darwin_amd64/dag
mv ./bazel-bin/alphasql/pipeline_type_checker ./alphasql_darwin_amd64/pipeline_type_checker
tar -zcvf bin/alphasql_darwin_amd64.tar.gz alphasql_darwin_amd64
tar -zcvf alphasql_darwin_amd64.tar.gz alphasql_darwin_amd64
- name: Get Release
id: get
uses: bruceadams/[email protected]
Expand All @@ -37,7 +37,7 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
upload_url: ${{ steps.get.outputs.upload_url }} # This pulls from the CREATE RELEASE step above, referencing it's ID to get its outputs object, which include a `upload_url`. See this blog post for more info: https://jasonet.co/posts/new-features-of-github-actions/#passing-data-to-future-steps
asset_path: ./bin/alphasql_darwin_amd64.tar.gz
asset_path: alphasql_darwin_amd64.tar.gz
asset_name: alphasql_darwin_amd64.tar.gz
asset_content_type: application/tar+gzip

Expand Down Expand Up @@ -72,7 +72,7 @@ jobs:
mkdir alphasql_linux_x86_64
mv ./bazel-bin/alphasql/dag ./alphasql_linux_x86_64/dag
mv ./bazel-bin/alphasql/pipeline_type_checker ./alphasql_linux_x86_64/pipeline_type_checker
tar -zcvf bin/alphasql_linux_x86_64.tar.gz alphasql_linux_x86_64
tar -zcvf alphasql_linux_x86_64.tar.gz alphasql_linux_x86_64
- name: Get Release
id: get
uses: bruceadams/[email protected]
Expand All @@ -83,7 +83,7 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
upload_url: ${{ steps.get.outputs.upload_url }} # This pulls from the CREATE RELEASE step above, referencing it's ID to get its outputs object, which include a `upload_url`. See this blog post for more info: https://jasonet.co/posts/new-features-of-github-actions/#passing-data-to-future-steps
asset_path: ./bin/alphasql_linux_x86_64.tar.gz
asset_path: alphasql_linux_x86_64.tar.gz
asset_name: alphasql_linux_x86_64.tar.gz
asset_content_type: application/tar+gzip

Expand Down
10 changes: 5 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -61,10 +61,10 @@ wget -P $temp https://github.com/Matts966/alphasql/releases/latest/download/alph

```bash
# To extract DAG from your SQL set
$ ./bin/osx/dag --output_path ./samples/sample1/dag.dot ./samples/sample1/
$ dag --output_path ./samples/sample1/dag.dot ./samples/sample1/

# Or you can check the output in stdout by
$ ./bin/osx/dag [paths]
$ dag [paths]

# with graphviz
$ dot -Tpng samples/sample1/dag.dot -o samples/sample1/dag.png
Expand Down Expand Up @@ -109,7 +109,7 @@ Note that you should run type_checker in the same path as in extracting DAG.

```bash
# to check type and schema of SQL set
$ ./bin/osx/pipeline_type_checker ./samples/sample1.dot
$ pipeline_type_checker ./samples/sample1.dot
Analyzing "./samples/sample1/create_datawarehouse3.sql"
DDL analyzed, adding table to catalog...
SUCCESS: analysis finished!
Expand Down Expand Up @@ -169,13 +169,13 @@ You can specify external schemata (not created by queries in SQL set) by passing

```bash
# with external schema
$ ./bin/osx/pipeline_type_checker --json_schema_path ./samples/sample-schema.json ./samples/sample1/dag.dot
$ pipeline_type_checker --json_schema_path ./samples/sample-schema.json ./samples/sample1/dag.dot
```

You can extract required external tables by

```bash
$ ./bin/osx/dag --external_required_tables_output_path ./required_tables.txt {./path/to/sqls}
$ dag --external_required_tables_output_path ./required_tables.txt {./path/to/sqls}
# and get schemata using bq command
$ cat ./required_tables.txt | while read line
do
Expand Down
6 changes: 0 additions & 6 deletions docker/linux-copy-bin.sh

This file was deleted.

0 comments on commit d7465c8

Please sign in to comment.