<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=ISO-8859-1"
 http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#ffffff">
Oops, I realize I misspoke.<br>
<br>
I would set up the columns with hook_install() / hook_uninstall(), but
would also add them to the schema with hook_schema_alter().<br>
<br>
On 10-07-12 12:30 PM, Brian Vuyk wrote:
<blockquote cite="mid:4C3B432E.7080500@brianvuyk.com" type="cite">
  <meta content="text/html; charset=ISO-8859-1"
 http-equiv="Content-Type">
I don't see any problem with it, although I would add / remove the
columns with hook_install() / hook_uninstall() rather than
hook_schema_alter().<br>
  <br>
Brian<br>
  <br>
On 10-07-12 12:19 PM, nan wich wrote:
  <blockquote cite="mid:705846.48166.qm@web180304.mail.gq1.yahoo.com"
 type="cite">
    <style type="text/css"><!-- DIV {margin:0px;} --></style>
    <div
 style="font-family: arial,helvetica,sans-serif; font-size: 12pt; color: rgb(127, 0, 63);">
    <div>Is there an official stance on using hook_schema_alter to
add&nbsp;columns to core tables? For example, we collect additional data on
anonymous comments and want to actually save that data. Rather than
creating another table (and subsequent JOINs), I'd just as soon stuff
that data into the&nbsp;comments table, where it belongs. Should we ever
disable comments (unlikely), we probably wouldn't mind losing that data
too.<br>
&nbsp;</div>
    <p><font face="bookman old style, new york, times, serif" size="4"
 color="#ff007f"><em><strong>Nancy E. Wichmann, PMP</strong></em></font></p>
    <p><font face="arial, helvetica, sans-serif">Injustice anywhere is
a
threat to justice everywhere. -- Dr. Martin L. King, Jr.</font></p>
    </div>
  </blockquote>
  <br>
</blockquote>
<br>
</body>
</html>