Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[NSE-576] Support from_unixtime expression in the case that 'yyyyMMdd' format is required #614

Merged
merged 3 commits into from
Dec 13, 2021

Conversation

PHILO-HE
Copy link
Collaborator

@PHILO-HE PHILO-HE commented Dec 8, 2021

This patch depends on oap-project/arrow#56.

@github-actions
Copy link

github-actions bot commented Dec 8, 2021

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/oap-project/native-sql-engine/issues

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

[NSE-${ISSUES_ID}] ${detailed message}

See also:

@@ -62,7 +62,7 @@ echo "ARROW_SOURCE_DIR=${ARROW_SOURCE_DIR}"
echo "ARROW_INSTALL_DIR=${ARROW_INSTALL_DIR}"
mkdir -p $ARROW_SOURCE_DIR
mkdir -p $ARROW_INSTALL_DIR
git clone https://github.com/oap-project/arrow.git --branch arrow-4.0.0-oap $ARROW_SOURCE_DIR
git clone https://github.com/philo-he/arrow.git --branch from_unixtime $ARROW_SOURCE_DIR
Copy link
Collaborator

Choose a reason for hiding this comment

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

can you remove this change?

@PHILO-HE
Copy link
Collaborator Author

The temp changes for arrow branch was just removed. Also rebased the code.

@zhouyuan zhouyuan changed the title Support from_unixtime expression in the case that 'yyyyMMdd' format is required [NSE-576] Support from_unixtime expression in the case that 'yyyyMMdd' format is required Dec 13, 2021
@github-actions
Copy link

#576

@zhouyuan zhouyuan merged commit 604d17b into oap-project:master Dec 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants