InstantForum.NET v2.9.0SQLServerCentralhttp://www.sqlservercentral.com/Forums/notifications@sqlservercentral.comTue, 21 May 2013 03:47:34 GMT20http://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspx[quote][b]Eugene Elutin (9/11/2012)[/b][hr]Actually the attribute splitting may not be a flaw, but done on purpose. One of the common places where you can see this kind of picture is data warehouses (non-conformed dimensions).[/quote]Or companies that vend solutions to multiple customers and can't mix their data for legal reasons. Like health insurance software companies that host the data on site. I actually worked for one of those.Of course, in that instance, any request to mingle the data via UNION (at least in the U.S.) would be a Bad Idea.Tue, 11 Sep 2012 08:09:15 GMTBrandie Tarvinhttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspx[quote][b]CELKO (9/11/2012)[/b][hr][quote] I have a database that contains a number of individual tables for each sales agents. [/quote]This design flaw is called attribute splitting. Would you have “Male_Personnel” and “Female_Personnel” tables or a single “Personnel” table? Pretty obvious, isn't it? ...[/quote]That is really bright idea! Actually, taking in count progress of the west civilization in whole, you need at least three tables:“Male_Personnel”, “Female_Personnel” and "Unknown_Personnel" :hehe:Actually the attribute splitting may not be a flaw, but done on purpose. One of the common places where you can see this kind of picture is data warehouses (non-conformed dimensions).There are other design patterns where attribute splitting is a feature - not a flow.Tue, 11 Sep 2012 07:55:04 GMTEugene Elutinhttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspx[quote][b]CELKO (9/11/2012)[/b][hr][quote]Comments deleted.[/quote]Really Mr. Celko? Your entire post is inappropriate, rude, and unprofessional. I personally would like to see you just leave this site and not come back.Tue, 11 Sep 2012 07:50:35 GMTLynn Pettishttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspxCelko,Cute joke.Now do you actually have advice that is any different from that which was already offered?Tue, 11 Sep 2012 07:49:50 GMTBrandie Tarvinhttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspx[quote] I have a database that contains a number of individual tables for each sales agents. [/quote]This design flaw is called attribute splitting. Would you have “Male_Personnel” and “Female_Personnel” tables or a single “Personnel” table? Pretty obvious, isn't it? As you just figured out, you will always be repairing this insanity with UNION ALL statements that have to be re-written every time a new employee is added. [quote] Any help would be greatly appreciated. [/quote]Now, you need to find the moron that did this to you and run him over in the parking lot. And finally, rewrite all of his code. That Agents is also a disaster. This is full of uselessly vague data element names. Record_type? This better not be meta data in RDBMS. Oh, minimal Netiquette is to post [i]actual DDL[/i] and not narrative lists. There is an Jewish joke in the garment district of New York City about "Levine the Tailor" who sells a guy a horrible suit. But if the buyer holds one pant leg up with his left hand, tilts his head, lifts a shoulder, etc. it will sort of fit. The guy does not like contorting himself like this, so Levine tells him to walk to the end of the block and if he does not get one compliment, the suit is free. With nothing to lose, he does it. When he gets to the corner, an old lady asks him "That is a great suit! Whose your tailor?" "Levine, in the middle of the block. Why did you ask?"The lady replies, "I don't mean to offend you, but if he can fit a suit to hopelessly deformed cripple like you, he can do anything with a suit!" Tue, 11 Sep 2012 07:43:15 GMTCELKOhttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspxcouldn't agree more!;-)Mon, 10 Sep 2012 07:30:52 GMTaaron.reesehttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspx[quote][b]aaron.reese (9/10/2012)[/b][hr]the view with UNION ALL the same query once for each agent table[/quote]Not something I recommend. The view will have to be recreated every time a new agents table is added, and it can break if the schema on any of the underlying tables is altered (except if WITH SCHEMABINDING is used, and that introduces its own issues).Plus the more UNIONs get added to a query, the worse the performance becomes. It gets really messy really quickly.Mon, 10 Sep 2012 07:23:56 GMTBrandie Tarvinhttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspxDBayliss - I looked at your other posts, and I think you've been landed with this task to maintain someone elses questionable code.Brandie's Idea (2), while not ideal, can be done as follows.First get the tables of type Agent - using the column names common to the Agent Tables as shown here (you can extend it to all the column names for greater accuracy)[code="sql"]SELECT DISTINCT T.TABLE_NAMEFROM INFORMATION_SCHEMA.TABLES AS TJOIN INFORMATION_SCHEMA.COLUMNS AS C ON C.TABLE_NAME = T.TABLE_NAMEWHERE C.COLUMN_NAME IN ( 'Start_Time' , 'End_Time', 'Record_Type', 'User_Action')[/code]Then Dynamically create a SELECT statement from each of these, UNIONing them all together I've only included a few of the columns in the select - as follows.Note: The fancy row_number bit is just to identify the last row, so it doesn't get a UNION ALL following. [code="sql"]DECLARE @SQL VARCHAR(MAX)SET @SQL = '';;WITH AgentTables AS ( SELECT DISTINCT T.TABLE_NAME FROM INFORMATION_SCHEMA.TABLES AS T JOIN INFORMATION_SCHEMA.COLUMNS AS C ON C.TABLE_NAME = T.TABLE_NAME WHERE C.COLUMN_NAME IN ( 'Start_Time' , 'End_Time', 'Record_Type', 'User_Action')), NumberedTables AS ( SELECT TABLE_NAME, ROW_NUMBER() OVER (ORDER BY TABLE_NAME) AS RowNum FROM AgentTables ) SELECT @SQL = @SQL + ' SELECT ''' + TABLE_NAME +''' AS AgentName, [Start_Time], [End_Time], [Record_Type] FROM ' + TABLE_NAME +CASE WHEN RowNum = (SELECT COUNT(*) FROM NumberedTables) THEN ''ELSE ' UNION ALL 'END FROM NumberedTablesORDER BY TABLE_NAME-- PRINT @SQL;EXEC (@SQL);[/code]This may get you out of a hole for now, but fixing the normalization is the way to go.Mon, 10 Sep 2012 06:52:24 GMTt.brown 89142http://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspxIf you created this database structure, please read up on database normalisation and correct the structure before you go any further down this design.If someone else created it and they are still around, go and puch them on the nose.If they are not around go and punch the wall.If you have to stick with this design, I would create a table of agents and a stored procedure that writes a view using dynamic SQL that is triggered every time the agents table is altered. the view with UNION ALL the same query once for each agent tableThe problem is that any other view or stored procedure that uses that generated view will also need to be recompiled every time the view changes.SERIOUSLY, get the data normalised.Mon, 10 Sep 2012 06:39:11 GMTaaron.reesehttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspxDBayliss, if you can follow t.brown's advice, that would be the best solution to your issue. If you are constrained from making changes to the existing schema, here are a couple of thoughts.1) Create a new table per t.brown's advice and have a job that inserts the new agent information when a new agent is added. Then use that to hit all the other tables. You could (though this would be poor relational programming) even include the agent's table name as a column in the agent's table.2) Loop through information_schema.tables and pull the table_name values from there. Regardless of how you dynamically pull these table names, you would have to have an idea of what separates these agent tables from all the other tables. Or you could just throw the agent table names into a brand new table (maybe even that table into a new DatabaseAdmin database) that you can pull the agent tables from. Use Dynamic SQL to do your UNION / UNION ALL.BTW, I don't really recommend this as your best option.3) Use a staging table to merge all the agent table data into, then query off of that. You could use any number of things to keep the staging table updated, like replication, or a regular LEFT OUTER JOIN type of INSERT statement, or even an SSIS package with slowly changing dimensions. The options depend on your data, really.Mon, 10 Sep 2012 06:07:17 GMTBrandie Tarvinhttp://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspxIt would be possible to build a dynamic query that checks for tablenames, and checks if those tables have the columns corresponding to an agent table, and if so includes them in the union, but it would be quite complex, and from an SQL Server point of view, like using a Ferrari to plough a field. Much better to use the relational database in the way it is designed to be used.Create an Agents_Name Table, with an ID columnAdd and AgentName_ID column to your Agent table.This way, all agents share the same Agent Table, but are identified by AgentName_IDThen instead of a union, you only query one table, possibly joining to the AgentName table to get the name if needed. Mon, 10 Sep 2012 03:20:33 GMTt.brown 89142http://www.sqlservercentral.com/Forums/Topic1356604-391-1.aspxI have a database that contains a number of individual tables for each sales agents. I need to find a way to set up a query that dynamically unions all tables in the database, so as an new agent joins the business their table will automatically be picked up in the union. The idea being, I have 1 table that contains the information for all agents.Any help would be greatly appreciated.Many thanks in advance.[b]Tables in database ( example)[/b]DavidJohnSteven[b]Columns in agents table ( example[/b]) ,[Start_Time] ,[End_Time] ,[Record_Type] ,[User_Action] ,[Direction] ,[Action_Data] ,[From_addr] ,[To_addr] ,[Subject] ,[Detail] ,[Company] ,[Contact] ,[FullPath]Mon, 10 Sep 2012 03:10:48 GMTDBayliss