Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CI #10

Merged
merged 12 commits into from
Apr 19, 2021
Merged

Fix CI #10

merged 12 commits into from
Apr 19, 2021

Conversation

andygrove
Copy link
Member

@andygrove andygrove commented Apr 18, 2021

Picking up from Jorge's PR #8

@github-actions
Copy link

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW

Opening JIRAs ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename pull request title in the following format?

ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

@andygrove
Copy link
Member Author

And now crates.io seems to be having issues

@andygrove
Copy link
Member Author

failed to get arrow data dir: the data dir `/Users/runner/work/arrow-datafusion/arrow-datafusion/testing/data` defined by env ARROW_TEST_DATA not found

@andygrove
Copy link
Member Author

It looks like I mistakenly added the arrow-testing submodule under arrow-testing instead of just testing

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@e717559). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 8ec2c7f differs from pull request most recent head ff3bcc8. Consider uploading reports for the commit ff3bcc8 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##             master      #10   +/-   ##
=========================================
  Coverage          ?   70.41%           
=========================================
  Files             ?      123           
  Lines             ?    21261           
  Branches          ?        0           
=========================================
  Hits              ?    14970           
  Misses            ?     6291           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e717559...ff3bcc8. Read the comment docs.

@andygrove
Copy link
Member Author

Rust tests pass. Travis is complaining about C++, Go, and Java.

@andygrove andygrove merged commit d2bc07d into apache:master Apr 19, 2021
houqp pushed a commit to jorgecarleitao/arrow-datafusion that referenced this pull request Sep 28, 2021
…ts (apache#10)

* Fix tests

* Ignore last test, fix clippy, fmt and enable integration

* more clippy fix
@andygrove andygrove deleted the fix-ci branch February 6, 2022 17:42
EricJoy2048 referenced this pull request in argoengine/arrow-datafusion Feb 24, 2022
EricJoy2048 referenced this pull request in argoengine/arrow-datafusion Feb 24, 2022
#10 Add event test data set

See merge request noah/argo_engine!12
EricJoy2048 referenced this pull request in argoengine/arrow-datafusion Feb 24, 2022
add parquet_uba_event&parquet_uba_profile to get uba test data dir in test_utils

Closes #10

See merge request noah/argo_engine!13
alamb added a commit that referenced this pull request Jul 16, 2024
… `interval` (#11466)

* Unparser rule for datatime cast (#10)

* use timestamp as the identifier for date64

* rename

* implement CustomDialectBuilder

* fix

* dialect with interval style (#11)

---------

Co-authored-by: Phillip LeBlanc <[email protected]>

* fmt

* clippy

* doc

* Update datafusion/sql/src/unparser/expr.rs

Co-authored-by: Andrew Lamb <[email protected]>

* update the doc for CustomDialectBuilder

* fix doc test

---------

Co-authored-by: Phillip LeBlanc <[email protected]>
Co-authored-by: Andrew Lamb <[email protected]>
xinlifoobar pushed a commit to xinlifoobar/datafusion that referenced this pull request Jul 17, 2024
… `interval` (apache#11466)

* Unparser rule for datatime cast (apache#10)

* use timestamp as the identifier for date64

* rename

* implement CustomDialectBuilder

* fix

* dialect with interval style (apache#11)

---------

Co-authored-by: Phillip LeBlanc <[email protected]>

* fmt

* clippy

* doc

* Update datafusion/sql/src/unparser/expr.rs

Co-authored-by: Andrew Lamb <[email protected]>

* update the doc for CustomDialectBuilder

* fix doc test

---------

Co-authored-by: Phillip LeBlanc <[email protected]>
Co-authored-by: Andrew Lamb <[email protected]>
xinlifoobar pushed a commit to xinlifoobar/datafusion that referenced this pull request Jul 18, 2024
… `interval` (apache#11466)

* Unparser rule for datatime cast (apache#10)

* use timestamp as the identifier for date64

* rename

* implement CustomDialectBuilder

* fix

* dialect with interval style (apache#11)

---------

Co-authored-by: Phillip LeBlanc <[email protected]>

* fmt

* clippy

* doc

* Update datafusion/sql/src/unparser/expr.rs

Co-authored-by: Andrew Lamb <[email protected]>

* update the doc for CustomDialectBuilder

* fix doc test

---------

Co-authored-by: Phillip LeBlanc <[email protected]>
Co-authored-by: Andrew Lamb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants