Skip to content

[To rel/1.1][IOTDB-5685] Fix error msg of failing to create a timeseries on an existing path when ReadOnly state#9354

Merged
MarcosZyk merged 2 commits intoapache:rel/1.1from
MarcosZyk:IOTDB-5685_1.1
Mar 17, 2023
Merged

[To rel/1.1][IOTDB-5685] Fix error msg of failing to create a timeseries on an existing path when ReadOnly state#9354
MarcosZyk merged 2 commits intoapache:rel/1.1from
MarcosZyk:IOTDB-5685_1.1

Conversation

@MarcosZyk
Copy link
Contributor

@MarcosZyk MarcosZyk commented Mar 16, 2023 •
edited
Loading

Description

The error msg thrown from consensus layer was not correctly returned to client. Fix it.

IoTDB> set system to readonly
Msg: The statement is executed successfully.
IoTDB> create timeseries root.db2.device2.s3 int32
Msg: 600: Fail to do non-query operations because system is read-only.

@MarcosZyk MarcosZyk changed the title [IOTDB-5685] Fix error msg of failing to create a timeseries on an existing path when ReadOnly state [To rel/1.1][IOTDB-5685] Fix error msg of failing to create a timeseries on an existing path when ReadOnly state Mar 16, 2023
CRZbulabula
CRZbulabula approved these changes Mar 17, 2023
View reviewed changes
Copy link
Contributor

@CRZbulabula CRZbulabula left a comment

Choose a reason for hiding this comment

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

LGTM～

SzyWilliam
SzyWilliam approved these changes Mar 17, 2023
View reviewed changes
Copy link
Member

@SzyWilliam SzyWilliam left a comment

Choose a reason for hiding this comment

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

+1 the changes look good to me.

@MarcosZyk MarcosZyk merged commit 4012ecb into apache:rel/1.1 Mar 17, 2023
@MarcosZyk MarcosZyk mentioned this pull request Mar 20, 2023
Merged
MarcosZyk added a commit that referenced this pull request Mar 20, 2023
@MarcosZyk
[To rel/1.1][IOTDB-5685] Fix error msg of failing to create a timeser…
2a23815 
…ies on an existing path when ReadOnly state (#9354)
@CritasWang CritasWang mentioned this pull request Apr 3, 2023
Merged
@MarcosZyk MarcosZyk deleted the IOTDB-5685_1.1 branch June 5, 2023 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CRZbulabula CRZbulabula CRZbulabula approved these changes

@SzyWilliam SzyWilliam SzyWilliam approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MarcosZyk @CRZbulabula @SzyWilliam