database name not included in RAW_SQL type in transaction_log

[Joe Daly]

doing the following

create table a (i int primary key)

results in the following in a Transaction message. If this were to be replicated it would cause a error as no database name is in the sql statement.

replicated message text=transaction_context {
  server_id: 1
  transaction_id: 41
  start_timestamp: 1266979792840979
  end_timestamp: 1266979792840991
}
statement {
  type: RAW_SQL
  start_timestamp: 1266979792840982
  end_timestamp: 1266979792840989
  sql: "create table a (i int primary key)"
}

[Jay Pipes (jaypipes)]

Yep, absolutely correct. I will work on this one. Slowly, RAW_SQL statements are being removed (and replaced with more specific message subclasses that better describe the statement) but in the meantime, it would be a pretty trivial fix to add in the schema name into the message.

Cheers!

Jay

[Brian Aker (brianaker)]

Hi!

It should be possible very soon to just send the schema proto over the
wire (and before April Table).

Cheers,
 -Brian
On Feb 24, 2010, at 6:26 AM, Jay Pipes wrote:

> Yep, absolutely correct. I will work on this one. Slowly, RAW_SQL
> statements are being removed (and replaced with more specific message
> subclasses that better describe the statement) but in the meantime, it
> would be a pretty trivial fix to add in the schema name into the
> message.
>
> Cheers!
>
> Jay
>
> ** Changed in: drizzle
> Status: New => Confirmed
>
> ** Changed in: drizzle
> Importance: Undecided => High
>
> ** Changed in: drizzle
> Assignee: (unassigned) => Jay Pipes (jaypipes)
>
> ** Changed in: drizzle
> Milestone: None => 2010-03-15
>
> --
> database name not included in RAW_SQL type in transaction_log
> https://bugs.launchpad.net/bugs/526812
> You received this bug notification because you are a member of
> Drizzle-
> developers, which is subscribed to Drizzle.
>
> Status in A Lightweight SQL Database for Cloud and Web: Confirmed
>
> Bug description:
> doing the following
>
> create table a (i int primary key)
>
> results in the following in a Transaction message. If this were to
> be replicated it would cause a error as no database name is in the
> sql statement.
>
> replicated message text=transaction_context {
> server_id: 1
> transaction_id: 41
> start_timestamp: 1266979792840979
> end_timestamp: 1266979792840991
> }
> statement {
> type: RAW_SQL
> start_timestamp: 1266979792840982
> end_timestamp: 1266979792840989
> sql: "create table a (i int primary key)"
> }
>
>

[Jay Pipes (jaypipes)]

Hi again, Joe.

OK, so I'm marking this won't fix b/c Brian actually has made excellent progress in the new schema engine, and it's actually easier now for me to start getting rid of RAW_SQL statements for most things, including the CREATE TABLE/ALTER TABLE statements.

Cheers,

Jay