Skip to content

Conversation

@Johnnyssc
Copy link
Contributor

If query execute in one table that is restoring data, query may failed with Exception occasionally.
Exception info :" Table state is not NORMAL: 'RESTORING'".

But mostly restoreJob only effects some partitions in this table, not all of them, so that the other partitions need support query successfully.

The reason why this problem happen occasionally is that Analyzer.resolveTableRef check olapTable.state and partition.state without readLock, so fix it by check partition.state only.

ps: set restore status for partitions in this pr:#8245

@doris-robot
Copy link

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.

@Johnnyssc Johnnyssc force-pushed the fix_restore_effect_query branch from 4eee036 to fc41ec8 Compare September 13, 2024 02:04
@Johnnyssc
Copy link
Contributor Author

run buildall

// if doing restore with table, throw exception here
ErrorReport.reportAnalysisException(ErrorCode.ERR_BAD_TABLE_STATE, "RESTORING");
}
Boolean isAllPartitionRestoring = ((OlapTable) table).getPartitions().stream()
Copy link
Contributor

Choose a reason for hiding this comment

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

legacy planner will be remove very soon, so u should impl it in Nereids

@github-actions
Copy link
Contributor

We're closing this PR because it hasn't been updated in a while.
This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and feel free a maintainer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Apr 10, 2025
@github-actions github-actions bot closed this Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants