<style>
/* Changing the layout to use less space for mobiles */
@media screen and (max-device-width: 480px), screen and (-webkit-min-device-pixel-ratio: 2) {
#email-body { min-width: 30em !important; }
#email-page { padding: 8px !important; }
#email-banner { padding: 8px 8px 0 8px !important; }
#email-avatar { margin: 1px 8px 8px 0 !important; padding: 0 !important; }
#email-fields { padding: 0 8px 8px 8px !important; }
#email-gutter { width: 0 !important; }
}
</style>
<div id="email-body">
<table id="email-wrap" align="center" border="0" cellpadding="0" cellspacing="0" style="background-color:#f0f0f0;color:#000000;width:100%;">
<tr valign="top">
<td id="email-page" style="padding:16px !important;">
<table align="center" border="0" cellpadding="0" cellspacing="0" style="background-color:#ffffff;border:1px solid #bbbbbb;color:#000000;width:100%;">
<tr valign="top">
<td bgcolor="#3e4c4e" style="background-color:#3e4c4e;color:#ffffff;font-family:Arial,FreeSans,Helvetica,sans-serif;font-size:12px;line-height:1;"><img src="https://www.jboss.org/dms/hibernate/images/jira/jiraheader_hibernate.png" alt="" style="vertical-align:top;" /></td>
</tr><tr valign="top">
<td id="email-banner" style="padding:32px 32px 0 32px;">
<table align="left" border="0" cellpadding="0" cellspacing="0" width="100%" style="width:100%;">
<tr valign="top">
<td style="color:#505050;font-family:Arial,FreeSans,Helvetica,sans-serif;padding:0;">
<img id="email-avatar" src="https://hibernate.onjira.com/secure/useravatar?ownerId=brmeyer&avatarId=11140" alt="" height="48" width="48" border="0" align="left" style="padding:0;margin: 0 16px 16px 0;" />
<div id="email-action" style="padding: 0 0 8px 0;font-size:12px;line-height:18px;">
<a class="user-hover" rel="brmeyer" id="email_brmeyer" href="https://hibernate.onjira.com/secure/ViewProfile.jspa?name=brmeyer" style="color:#6c797f;">Brett Meyer</a>
commented on <img src="https://hibernate.onjira.com/images/icons/bug.gif" height="16" width="16" border="0" align="absmiddle" alt="Bug"> <a style='color:#6c797f;text-decoration:none;' href='https://hibernate.onjira.com/browse/HHH-8021'>HHH-8021</a>
</div>
<div id="email-summary" style="font-size:16px;line-height:20px;padding:2px 0 16px 0;">
<a style='color:#6c797f;text-decoration:none;' href='https://hibernate.onjira.com/browse/HHH-8021'><strong>Overriding the table for columns in nested embeddable classes</strong></a>
</div>
</td>
</tr>
</table>
</td>
</tr>
<tr valign="top">
<td id="email-fields" style="padding:0 32px 32px 32px;">
<table border="0" cellpadding="0" cellspacing="0" style="padding:0;text-align:left;width:100%;" width="100%">
<tr valign="top">
<td id="email-gutter" style="width:64px;white-space:nowrap;"></td>
<td>
<table border="0" cellpadding="0" cellspacing="0" width="100%">
<tr valign="top">
<td colspan="2" style="color:#000000;font-family:Arial,FreeSans,Helvetica,sans-serif;font-size:12px;padding:0 0 16px 0;width:100%;">
<div class="comment-block" style="background-color:#edf5ff;border:1px solid #dddddd;color:#000000;padding:12px;"><blockquote>
<p>(10:32:27 AM) sebersole: i had a quick look and because of the tight coupling of the notion of Value in the old model code, i dont think the org.hibernate.mapping.Component class supports this at all<br/>
(10:32:51 AM) sebersole: Value is assumed to come all from a table<br/>
(10:33:00 AM) sebersole: and org.hibernate.mapping.Component extends Value<br/>
(10:35:16 AM) sebersole: anyway, my point is that this would require good bit of work for 4.x code<br/>
(10:35:44 AM) sebersole: imo you'd need to re-work the org.hibernate.mapping.Component definition<br/>
(10:41:31 AM) sebersole: one option might be to add getTable to Column and by default have it look to its value.getTable<br/>
(10:41:50 AM) sebersole: but add the ability to override it "locally" to the column<br/>
...<br/>
(10:50:12 AM) sebersole: you cant know, given a component, that its columns come from different tables<br/>
(10:50:36 AM) sebersole: the old model does not support that, and was never changed to support that<br/>
(10:51:11 AM) sebersole: one way is a pointer from Column back to Table<br/>
(10:51:34 AM) sebersole: but that a new api method<br/>
...<br/>
(10:54:42 AM) sebersole: tbh, not sure if it is just the nesting that causes this<br/>
(10:56:17 AM) sebersole: meaning i think that may just be anecdotal<br/>
(10:58:57 AM) emmanuel: sebersole: it's possible as you said that an embeddable cannot have columns form two tables. so the test case could be simplified.<br/>
(11:01:36 AM) sebersole: emmanuel: do you know if jpa says one way or another whether embeddables can be split this way?<br/>
(11:01:50 AM) sebersole: split across tables i mean<br/>
(11:02:22 AM) emmanuel: it does not say so it implies support IMO. But I doubt we will ever see a test in the TCK <img class="emoticon" src="https://hibernate.onjira.com/images/icons/emoticons/smile.gif" height="20" width="20" align="absmiddle" alt="" border="0"/><br/>
(11:02:36 AM) emmanuel: It's one of those legacy mapping that would be nice to have<br/>
(11:03:14 AM) sebersole: i agree it would be nice to have<br/>
(11:03:35 AM) sebersole: just as it is i dont see a requirement that it be in 4.x<br/>
(11:04:45 AM) emmanuel: sebersole: same feeling<br/>
(11:04:49 AM) sebersole: adding support for this (at least via column knowing about its table) makes the table structure no longer acyclic<br/>
(11:05:10 AM) sebersole: as we'd have pointers both directions<br/>
(11:05:30 AM) emmanuel: sebersole: do you have to make a link, after all we have secondary table support alreadya nd it does not require this link<br/>
(11:07:42 AM) sebersole: not following what you mean<br/>
(11:08:02 AM) sebersole: we do in fact have that in secondary table support<br/>
(11:08:28 AM) sebersole: secondary table maps to a class named Join<br/>
(11:08:35 AM) sebersole: because hbm called this <join/><br/>
(11:09:06 AM) sebersole: the Join itself carries the join key info<br/>
(11:09:36 AM) sebersole: Join always assumes pk joinage<br/>
(11:10:37 AM) sebersole: again the problem really is Value<br/>
(11:10:58 AM) sebersole: most things you map a Type to do not "span" tables<br/>
(11:11:16 AM) sebersole: so Value is currently the value at which table is mapped<br/>
(11:11:32 AM) sebersole: so in a way, we do already in fact have this cyclic link<br/>
(11:11:43 AM) sebersole: 1) table knows about columns<br/>
(11:12:00 AM) sebersole: 2) column knows about value which knows about table<br/>
(11:12:24 AM) sebersole: the problem is that Value (here the Component) can only name one table<br/>
(11:13:54 AM) sebersole: its a modeling problem (imo) in o.h.mapping with Value<br/>
(11:14:11 AM) sebersole: and specifically with all the things that try to be a Value</p></blockquote></div>
<div style="color:#505050;padding:4px 0 0 0;"> </div>
</td>
</tr>
</table>
</td>
</tr>
</table>
</td>
</tr>
</table>
</td><!-- End #email-page -->
</tr>
<tr valign="top">
<td style="color:#505050;font-family:Arial,FreeSans,Helvetica,sans-serif;font-size:10px;line-height:14px;padding: 0 16px 16px 16px;text-align:center;">
This message is automatically generated by JIRA.<br />
If you think it was sent incorrectly, please contact your JIRA administrators<br />
For more information on JIRA, see: <a style='color:#6c797f;' href='http://www.atlassian.com/software/jira'>http://www.atlassian.com/software/jira</a>
</td>
</tr>
</table><!-- End #email-wrap -->
</div><!-- End #email-body -->