Skip to content

[AMORO-3091] Support TIMESTAMP in MixedDataFiles.fromPartitionString()#3094

Merged
zhoujinsong merged 4 commits intoapache:masterfrom
lurnagao-dahua:fix_issues_3091
Aug 13, 2024
Merged

[AMORO-3091] Support TIMESTAMP in MixedDataFiles.fromPartitionString()#3094
zhoujinsong merged 4 commits intoapache:masterfrom
lurnagao-dahua:fix_issues_3091

Conversation

@lurnagao-dahua
Copy link
Contributor

fix issues 3091
support TIMESTAMP in MixedDataFiles.fromPartitionString()

@github-actions github-actions bot added the module:core Core module label Aug 12, 2024
@czy006
Copy link
Contributor

czy006 commented Aug 13, 2024

Thank you for your First Contribution,We have a set of template standards when we submit PR, and I revised some of the descriptions.and you should run spotless format the code. cc @lurnagao-dahua
image

@huyuanfeng2018 huyuanfeng2018 changed the title Fix issues 3091 [AMORO-3901] Support TIMESTAMP in MixedDataFiles.fromPartitionString() Aug 13, 2024
@huyuanfeng2018 huyuanfeng2018 changed the title [AMORO-3901] Support TIMESTAMP in MixedDataFiles.fromPartitionString() [AMORO-3091] Support TIMESTAMP in MixedDataFiles.fromPartitionString() Aug 13, 2024
@lurnagao-dahua
Copy link
Contributor Author

lurnagao-dahua commented Aug 13, 2024

Thank you for your First Contribution,We have a set of template standards when we submit PR, and I revised some of the descriptions.and you should run spotless format the code. cc @lurnagao-dahua

Thank you for your guidance!
I have run mvn spotless:check and mvn spotless:apply

@zhoujinsong zhoujinsong linked an issue Aug 13, 2024 that may be closed by this pull request
2 tasks
Copy link
Contributor

@zhoujinsong zhoujinsong left a comment

Choose a reason for hiding this comment

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

LGTM.

Thanks for the contribution!

@zhoujinsong zhoujinsong merged commit da62faf into apache:master Aug 13, 2024
@lurnagao-dahua
Copy link
Contributor Author

Thanks for the review and merge @zhoujinsong!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:core Core module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Error occurred while viewing partitioned files of timestamp type.

3 participants