[Enhance](DOE)Support parse default es iso datetime string #17412

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

stalary merged 4 commits into apache:master from SaintBacchus:EsIsotime

Mar 10, 2023

Contributor

SaintBacchus commented Mar 4, 2023 •

edited

Loading

Proposed changes

Issue Number: close #xxx

Problem summary

Using cctz to parse default es iso datetime string;

Checklist(Required)

Does it affect the original behavior
Has unit tests been added
Has document been added or modified
Does it need to update dependencies
Is this PR support rollback (If NO, please explain WHY)

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

github-actions bot added the kind/test label

Contributor Author

SaintBacchus commented Mar 4, 2023

run buildall

Contributor

github-actions bot commented Mar 4, 2023

clang-tidy review says "All clean, LGTM! 👍"

morningman added the dev/1.2.3 label

stalary reviewed

View reviewed changes

docker/thirdparties/custom_settings.env Outdated Show resolved Hide resolved

fe/fe-core/src/main/java/org/apache/doris/external/elasticsearch/EsUtil.java Outdated Show resolved Hide resolved

regression-test/conf/regression-conf.groovy Outdated Show resolved Hide resolved

Contributor

github-actions bot commented Mar 8, 2023

clang-tidy review says "All clean, LGTM! 👍"

3 similar comments

Contributor

github-actions bot commented Mar 8, 2023

clang-tidy review says "All clean, LGTM! 👍"

Contributor

github-actions bot commented Mar 9, 2023

clang-tidy review says "All clean, LGTM! 👍"

Contributor

github-actions bot commented Mar 9, 2023

clang-tidy review says "All clean, LGTM! 👍"

SaintBacchus added 2 commits

March 9, 2023 15:20


          support parse default es iso datetime string

a746795


          data4

3d32bdd

Contributor

github-actions bot commented Mar 9, 2023

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment

Contributor

github-actions bot commented Mar 9, 2023

clang-tidy review says "All clean, LGTM! 👍"


          fix date

6d6a5bc

Contributor

github-actions bot commented Mar 9, 2023

clang-tidy review says "All clean, LGTM! 👍"


          fix datetime for es

42e50af

Contributor

github-actions bot commented Mar 9, 2023

clang-tidy review says "All clean, LGTM! 👍"

stalary approved these changes

View reviewed changes

Contributor

stalary left a comment

LGTM

Contributor Author

SaintBacchus commented Mar 9, 2023

run buildall

github-actions bot added the approved label

Contributor

github-actions bot commented Mar 9, 2023

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

Contributor

github-actions bot commented Mar 9, 2023

PR approved by anyone and no changes requested.

github-actions bot added the reviewed label

Contributor Author

SaintBacchus commented Mar 9, 2023

run p0

3 similar comments

Contributor Author

SaintBacchus commented Mar 9, 2023

run p0

Contributor Author

SaintBacchus commented Mar 9, 2023

run p0

Contributor

stalary commented Mar 9, 2023

run p0

stalary merged commit 4ba93ef into apache:master

gnehil pushed a commit to gnehil/doris that referenced this pull request


          [Enhance](DOE)Support parse default es iso datetime string (apache#17412

c711e49

)

* support parse default es iso datetime string

qidaye pushed a commit to qidaye/incubator-doris that referenced this pull request


          [Enhance](DOE)Support parse default es iso datetime string (apache#17412

8f04edb

)

* support parse default es iso datetime string

qidaye mentioned this pull request

[pick](DOE) pick doe fixes from master to branch-1.2-lts #22685

Merged

morningman pushed a commit that referenced this pull request


          [pick](DOE) pick doe fixes from master to branch-1.2-lts (#22685)

2bcf6c2

PRs:
1. #17566 
2. #17412 
3. #17745 
4. #22279 
5. #22046 
6. #22423

GoGoWen pushed a commit to GoGoWen/incubator-doris that referenced this pull request


          [pick](DOE) pick doe fixes from master to branch-1.2-lts (apache#22685)

9bc5fbf

PRs:
1. apache#17566 
2. apache#17412 
3. apache#17745 
4. apache#22279 
5. apache#22046 
6. apache#22423

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

Labels

approved dev/1.2.3 kind/test reviewed