Skip to content

[regression-test](parquet) add some parquet cases with tvf #40379

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.

Sign up for GitHub

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

Jump to bottom
Merged
morningman merged 10 commits into from
Sep 27, 2024
Merged

[regression-test](parquet) add some parquet cases with tvf #40379

morningman merged 10 commits into from
Sep 27, 2024

Conversation

@suxiaogang223
Copy link
Contributor

@suxiaogang223 suxiaogang223 commented Sep 4, 2024 •
edited
Loading

Add some parquet file as tvf case

@doris-robot
Copy link

doris-robot commented Sep 4, 2024

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@suxiaogang223 suxiaogang223 marked this pull request as draft September 4, 2024 11:25
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2024

Possible file(s) that should be tracked in LFS detected: 🚨

The following file(s) exceeds the file size limit: 1048576 bytes, as set in the .yml configuration files:

  • regression-test/data/external_table_p0/tvf/test_hdfs_parquet_group6.out

Consider using git-lfs to manage large files.

@github-actions github-actions bot added the lfs-detected! Warning Label for use when LFS is detected in the commits of a Pull Request label Sep 4, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Sep 20, 2024

Possible file(s) that should be tracked in LFS detected: 🚨

The following file(s) exceeds the file size limit: 1048576 bytes, as set in the .yml configuration files:

  • regression-test/data/external_table_p0/tvf/test_hdfs_parquet_group6.out

Consider using git-lfs to manage large files.

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Sep 20, 2024

Possible file(s) that should be tracked in LFS detected: 🚨

The following file(s) exceeds the file size limit: 1048576 bytes, as set in the .yml configuration files:

  • regression-test/data/external_table_p0/tvf/test_hdfs_parquet_group6.out

Consider using git-lfs to manage large files.

@github-actions github-actions bot removed the lfs-detected! Warning Label for use when LFS is detected in the commits of a Pull Request label Sep 20, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Sep 22, 2024

Possible file(s) that should be tracked in LFS detected: 🚨

The following file(s) exceeds the file size limit: 1048576 bytes, as set in the .yml configuration files:

  • regression-test/data/external_table_p0/tvf/test_hdfs_parquet_group5.out

Consider using git-lfs to manage large files.

@github-actions github-actions bot added the lfs-detected! Warning Label for use when LFS is detected in the commits of a Pull Request label Sep 22, 2024
@github-actions github-actions bot removed the lfs-detected! Warning Label for use when LFS is detected in the commits of a Pull Request label Sep 22, 2024
@suxiaogang223 suxiaogang223 marked this pull request as ready for review September 24, 2024 08:37
@suxiaogang223
Copy link
Contributor Author

suxiaogang223 commented Sep 24, 2024

run external

@suxiaogang223
Copy link
Contributor Author

suxiaogang223 commented Sep 24, 2024

run external

@suxiaogang223
Copy link
Contributor Author

suxiaogang223 commented Sep 25, 2024

run buildall

@suxiaogang223
Copy link
Contributor Author

suxiaogang223 commented Sep 25, 2024

run external

@suxiaogang223
Copy link
Contributor Author

suxiaogang223 commented Sep 25, 2024

run buildall

@suxiaogang223
Copy link
Contributor Author

suxiaogang223 commented Sep 25, 2024

run buildall

@github-actions
Copy link
Contributor

github-actions bot commented Sep 26, 2024

PR approved by anyone and no changes requested.

morningman
morningman approved these changes Sep 26, 2024
View reviewed changes
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Sep 26, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Sep 26, 2024

PR approved by at least one committer and no changes requested.

morningman
morningman approved these changes Sep 26, 2024
View reviewed changes
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morningman morningman merged commit 088d5dc into apache:master Sep 27, 2024
@suxiaogang223 suxiaogang223 deleted the test_hdfs_parquet branch September 27, 2024 09:16
gavinchou pushed a commit that referenced this pull request Sep 28, 2024
@suxiaogang223 @gavinchou
[regression-test](parquet) add some parquet cases with tvf (#40379)
025b7ee 
Add some parquet file as tvf case
cjj2010 pushed a commit to cjj2010/doris that referenced this pull request Oct 12, 2024
@suxiaogang223 @cjj2010
[regression-test](parquet) add some parquet cases with tvf (apache#40379
3503884 
)

Add some parquet file as tvf case
suxiaogang223 added a commit to suxiaogang223/doris that referenced this pull request Oct 16, 2024
@suxiaogang223
[regression-test](parquet) add some parquet cases with tvf (apache#40379
34ef067 
)

Add some parquet file as tvf case
@suxiaogang223 suxiaogang223 mentioned this pull request Oct 21, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morningman morningman morningman approved these changes

@wuwenchi wuwenchi wuwenchi approved these changes

+1 more reviewer

@zgxme zgxme zgxme approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by one committer. dev/2.1.7-merged dev/3.0.2-merged p0_b reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@suxiaogang223 @doris-robot @morningman @wuwenchi @zgxme @gavinchou