PostgreSQL insert many with returning

[billy1624]

PR Info

• Closes Inserting active models by insert_many with on_conflict and do_nothing panics if no rows are inserted. #899

Breaking Changes

• PostgreSQL insert many with returning - throw DbErr::RecordNotInserted if none of the records are being inserted

[tyt2y3]

upsert

[tyt2y3]

Thank you. Since #1002 has been merged, I'd like this to nest under the RuntimeError as discussed.

[billy1624]

Thank you. Since #1002 has been merged, I'd like this to nest under the RuntimeError as discussed.

I think some of the errors that reside at the DbErr root should be a variant of RuntimeError as well, such as UnpackInsertId and UpdateGetPrimaryKey.

So, to be consistent I'd like RecordNotInserted to be consistent with those.

sea-orm/src/error.rs

Lines 5 to 75 in 9d0f3f6

	/// An error from unsuccessful database operations
	#[derive(Error, Debug)]
	pub enum DbErr {
	/// This error can happen when the connection pool is fully-utilized
	#[error("Failed to acquire connection from pool")]
	ConnectionAcquire,
	/// Runtime type conversion error
	#[error("Error converting `{from}` into `{into}`: {source}")]
	TryIntoErr {
	/// From type
	from: &'static str,
	/// Into type
	into: &'static str,
	/// TryError
	source: Box<dyn std::error::Error + Send + Sync>,
	},
	/// There was a problem with the database connection
	#[error("Connection Error: {0}")]
	Conn(#[source] RuntimeErr),
	/// An operation did not execute successfully
	#[error("Execution Error: {0}")]
	Exec(#[source] RuntimeErr),
	/// An error occurred while performing a query
	#[error("Query Error: {0}")]
	Query(#[source] RuntimeErr),
	/// Type error: the specified type cannot be converted from u64. This is not a runtime error.
	#[error("Type '{0}' cannot be converted from u64")]
	ConvertFromU64(&'static str),
	/// After an insert statement it was impossible to retrieve the last_insert_id
	#[error("Failed to unpack last_insert_id")]
	UnpackInsertId,
	/// When updating, a model should know its primary key to check
	/// if the record has been correctly updated, otherwise this error will occur
	#[error("Failed to get primary key from model")]
	UpdateGetPrimaryKey,
	/// The record was not found in the database
	#[error("RecordNotFound Error: {0}")]
	RecordNotFound(String),
	/// Thrown by `TryFrom<ActiveModel>`, which assumes all attributes are set/unchanged
	#[error("Attribute {0} is NotSet")]
	AttrNotSet(String),
	/// A custom error
	#[error("Custom Error: {0}")]
	Custom(String),
	/// Error occurred while parsing value as target type
	#[error("Type Error: {0}")]
	Type(String),
	/// Error occurred while parsing json value as target type
	#[error("Json Error: {0}")]
	Json(String),
	/// A migration error
	#[error("Migration Error: {0}")]
	Migration(String),
	/// None of the records are being inserted into the database,
	/// if you insert with upsert expression that means
	/// all of them conflict with existing records in the database
	#[error("RecordNotInserted Error: {0}")]
	RecordNotInserted(String),
	}
	/// Runtime error
	#[derive(Error, Debug)]
	pub enum RuntimeErr {
	/// SQLx Error
	#[cfg(feature = "sqlx-dep")]
	#[error("{0}")]
	SqlxError(SqlxError),
	/// Error generated from within SeaORM
	#[error("{0}")]
	Internal(String),
	}

[tyt2y3]

Actually after a second thought I think it all makes sense.
Because there is no SQLx error at all (if there is, we'd like to keep the source)