Skip to content

[minor](fe) optimize some log info and imports issue - #24138

Merged
yiguolei merged 3 commits into
apache:masterfrom
morningman:remove_lists
Sep 11, 2023
Merged

[minor](fe) optimize some log info and imports issue#24138
yiguolei merged 3 commits into
apache:masterfrom
morningman:remove_lists

Conversation

@morningman

@morningmanmorningman commented Sep 9, 2023•
edited
Loading

Copy link
Copy Markdown
Contributor
  1. When setting cluster_id, give some detail instruction
  2. Disallow org.apache.hadoop.util.Lists
  3. fix when enable bdbje_debug_mode, NPE will be thrown when starting FE

@morningman

morningman commented Sep 9, 2023

Copy link
Copy Markdown
ContributorAuthor

run buildall

@doris-robot

doris-robot commented Sep 9, 2023

Copy link
Copy Markdown

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47 seconds
stream load tsv: 546 seconds loaded 74807831229 Bytes, about 130 MB/s
stream load json: 20 seconds loaded 2358488459 Bytes, about 112 MB/s
stream load orc: 64 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select: 28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size: 17161940315 Bytes

@morningman

morningman commented Sep 10, 2023

Copy link
Copy Markdown
ContributorAuthor

run buildall

@doris-robot

doris-robot commented Sep 10, 2023

Copy link
Copy Markdown

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.72 seconds
stream load tsv: 582 seconds loaded 74807831229 Bytes, about 122 MB/s
stream load json: 21 seconds loaded 2358488459 Bytes, about 107 MB/s
stream load orc: 64 seconds loaded 1101869774 Bytes, about 16 MB/s
stream load parquet: 31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select: 29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size: 17162185193 Bytes

yiguolei
yiguolei approved these changes Sep 11, 2023
View reviewed changes

@yiguoleiyiguolei left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Sep 11, 2023
@github-actions

github-actionsBot commented Sep 11, 2023

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actionsBot commented Sep 11, 2023

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

kaka11chen
kaka11chen approved these changes Sep 11, 2023
View reviewed changes

@kaka11chenkaka11chen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@yiguolei
yiguolei merged commit 6384198 into apache:masterSep 11, 2023
xiaokang pushed a commit that referenced this pull request Sep 11, 2023
@morningman@xiaokang
[minor](fe) optimize some log info and imports issue (#24138)
fcf5092
xiaokang pushed a commit that referenced this pull request Sep 13, 2023
@morningman@xiaokang
[minor](fe) optimize some log info and imports issue (#24138)
832ee7b
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kaka11chenkaka11chenkaka11chen approved these changes
@yiguoleiyiguoleiyiguolei approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.dev/2.0.2-mergedreviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@morningman@doris-robot@kaka11chen@yiguolei@xiaokang