Skip to content

Commit fab39fe

Browse files
Rename OperatorSignature to DropOperatorSignature and update references
1 parent c047b3f commit fab39fe

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed

src/ast/ddl.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4198,7 +4198,7 @@ pub struct DropOperator {
41984198
/// `IF EXISTS` clause
41994199
pub if_exists: bool,
42004200
/// One or more operators to drop with their signatures
4201-
pub operators: Vec<OperatorSignature>,
4201+
pub operators: Vec<DropOperatorSignature>,
42024202
/// `CASCADE or RESTRICT`
42034203
pub drop_behavior: Option<DropBehavior>,
42044204
}
@@ -4207,7 +4207,7 @@ pub struct DropOperator {
42074207
#[derive(Debug, Clone, PartialEq, PartialOrd, Eq, Ord, Hash)]
42084208
#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
42094209
#[cfg_attr(feature = "visitor", derive(Visit, VisitMut))]
4210-
pub struct OperatorSignature {
4210+
pub struct DropOperatorSignature {
42114211
/// Operator name
42124212
pub name: ObjectName,
42134213
/// Left operand type
@@ -4216,7 +4216,7 @@ pub struct OperatorSignature {
42164216
pub right_type: DataType,
42174217
}
42184218

4219-
impl fmt::Display for OperatorSignature {
4219+
impl fmt::Display for DropOperatorSignature {
42204220
fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
42214221
write!(f, "{} (", self.name)?;
42224222
if let Some(left_type) = &self.left_type {

src/ast/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ pub use self::ddl::{
7171
GeneratedExpressionMode, IdentityParameters, IdentityProperty, IdentityPropertyFormatKind,
7272
IdentityPropertyKind, IdentityPropertyOrder, IndexColumn, IndexOption, IndexType,
7373
KeyOrIndexDisplay, Msck, NullsDistinctOption, OperatorArgTypes, OperatorClassItem,
74-
OperatorPurpose, OperatorSignature, Owner, Partition, ProcedureParam, ReferentialAction,
74+
OperatorPurpose, DropOperatorSignature, Owner, Partition, ProcedureParam, ReferentialAction,
7575
RenameTableNameKind, ReplicaIdentity, TagsColumnOption, TriggerObjectKind, Truncate,
7676
UserDefinedTypeCompositeAttributeDef, UserDefinedTypeInternalLength,
7777
UserDefinedTypeRangeOption, UserDefinedTypeRepresentation, UserDefinedTypeSqlDefinitionOption,

src/parser/mod.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7542,7 +7542,7 @@ impl<'a> Parser<'a> {
75427542

75437543
/// Parse an operator signature for a [Statement::DropOperator]
75447544
/// Format: `name ( { left_type | NONE } , right_type )`
7545-
fn parse_operator_signature(&mut self) -> Result<OperatorSignature, ParserError> {
7545+
fn parse_operator_signature(&mut self) -> Result<DropOperatorSignature, ParserError> {
75467546
let name = self.parse_operator_name()?;
75477547
self.expect_token(&Token::LParen)?;
75487548

@@ -7560,7 +7560,7 @@ impl<'a> Parser<'a> {
75607560

75617561
self.expect_token(&Token::RParen)?;
75627562

7563-
Ok(OperatorSignature {
7563+
Ok(DropOperatorSignature {
75647564
name,
75657565
left_type,
75667566
right_type,

tests/sqlparser_postgres.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6765,15 +6765,15 @@ fn parse_create_operator() {
67656765

67666766
#[test]
67676767
fn parse_drop_operator() {
6768-
use sqlparser::ast::{DataType, DropBehavior, DropOperator, OperatorSignature};
6768+
use sqlparser::ast::{DataType, DropBehavior, DropOperator, DropOperatorSignature};
67696769

67706770
// Test DROP OPERATOR with NONE for prefix operator
67716771
let sql = "DROP OPERATOR ~ (NONE, BIT)";
67726772
assert_eq!(
67736773
pg().verified_stmt(sql),
67746774
Statement::DropOperator(DropOperator {
67756775
if_exists: false,
6776-
operators: vec![OperatorSignature {
6776+
operators: vec![DropOperatorSignature {
67776777
name: ObjectName::from(vec![Ident::new("~")]),
67786778
left_type: None,
67796779
right_type: DataType::Bit(None),
@@ -6801,7 +6801,7 @@ fn parse_drop_operator() {
68016801
pg().verified_stmt(&sql),
68026802
Statement::DropOperator(DropOperator {
68036803
if_exists: if_exist,
6804-
operators: vec![OperatorSignature {
6804+
operators: vec![DropOperatorSignature {
68056805
name: ObjectName::from(vec![Ident::new(*op)]),
68066806
left_type: Some(DataType::Integer(None)),
68076807
right_type: DataType::Integer(None),
@@ -6819,7 +6819,7 @@ fn parse_drop_operator() {
68196819
pg().verified_stmt(sql),
68206820
Statement::DropOperator(DropOperator {
68216821
if_exists: false,
6822-
operators: vec![OperatorSignature {
6822+
operators: vec![DropOperatorSignature {
68236823
name: ObjectName::from(vec![Ident::new("myschema"), Ident::new("@@")]),
68246824
left_type: Some(DataType::Text),
68256825
right_type: DataType::Text,
@@ -6835,12 +6835,12 @@ fn parse_drop_operator() {
68356835
Statement::DropOperator(DropOperator {
68366836
if_exists: true,
68376837
operators: vec![
6838-
OperatorSignature {
6838+
DropOperatorSignature {
68396839
name: ObjectName::from(vec![Ident::new("+")]),
68406840
left_type: Some(DataType::Integer(None)),
68416841
right_type: DataType::Integer(None),
68426842
},
6843-
OperatorSignature {
6843+
DropOperatorSignature {
68446844
name: ObjectName::from(vec![Ident::new("-")]),
68456845
left_type: Some(DataType::Integer(None)),
68466846
right_type: DataType::Integer(None),

0 commit comments

Comments
 (0)