[Okta Webinar] Learn how to a build a cloud-first strategyRegister Now

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 219
  • Last Modified:

Join vs. Where for speed - help me explain why I do it!

I'm in a bit of a bind.  I've been designing databases for quite a while and I've always tended to use joins to restrict data rather than having a bunch of indexes in the same table.

Currently, one of my clients hired their own "consultant", who's been nagging me about every design decision and it's driving me crazy trying to explain "why" I do things.

Here is the most recent example:

Our inventory system does a lot of crazy analysis on stock, but some products are just unimportant.  For example, You don't need to do a lot of analysis on office supplies when you're tracking the inventory of o a factory that makes cameras, even if they're in the same table.

I have a table where I have product_features, which simply has a product_id and a feature code.  When I save the product, I insert a record in the product features table that says, product_id, "trackstock".  Then, when I want to pull the inventory for an analysis report, I start with a view like this:  

select p.* from products p inner join product_features on product_id = 123 and feature_code='trackstock'


however, the obvious alternative would be to add a bit column in the products table for track_stock, so you could just do this:

select p.* from products p where track_stock=1

In regards to load, this table is going to have 5 million + records in it.  I'm mostly concerned that we're going to end up with a table that's got 100 columns

In this case, I'd say the guy has a point, and I told him to go ahead and make yet another new column, and another new index on that table, but honestly, am I crazy?  Is my constant use of joins to restrict data just a bizarre habit, or is there some justification to it??

Another example of my use that he disagrees with is for security.  I have procedures where you send a request for orders, but you only want to get the orders from the offices where that user has access, so instead of doing:

select * from orders where office_id in (select office_id from office_security where user_id=123)  - and a bunch of other ifs if the guy is an admin...

I actually use a table function so I can do this:

select o.* from orders o inner join security(@user_id) sec on sec.office_id = o.office_id

I dunno.  Maybe I'm crazy.  Maybe I'm sticking to my guns so hard because I'm not used to someone inexperienced, yet tells everyone he's a genius, questioning my every decision.

anyone with recommendations for my sanity?
0
Danielcmorris
Asked:
Danielcmorris
  • 3
  • 3
1 Solution
 
Kent OlsenData Warehouse Architect / DBACommented:
Hi Daniel,

Based on your descriptions, I'm on your side on both of these things.

Your "product_features" table is a form of normalization that simply removes this item from the table.  It allows you to easily define multiple features for any product.

The security test is absolutely the right way to go.  The host (database) controls the query and the results.  This has the least chance of SQL injection or hacking to obtain unauthorized information.


Good Luck,
Kent
0
 
DanielcmorrisAuthor Commented:
thanks Kent.  

I wish you could have been in the room when I got politically outmaneuvered by this smarmy consultant.
0
 
Kent OlsenData Warehouse Architect / DBACommented:
Yeah.  Me, too.  I don't get outmaneuvered often.  Especially by puppies.

I'll be glad to offer a more official 2 cents if it helps.


Kent
0
Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
DanielcmorrisAuthor Commented:
I shouldn't get so pissy.  I actually own the code, and they've put over 250k in modifications that are resellable.  

Still, I just don't know how someone gets to be CEO and be so gullible.  

Maybe we should get a few decent developers together and write a For Dummies book titled: "How to spot a corporate consultant who's full of crap"
0
 
Kent OlsenData Warehouse Architect / DBACommented:
You don't have to know a thing about technology to be a CEO.  But you DO have to know how to run a business.  One organization that I work with actually has a CIO that is a technology neophyte.

>> Maybe we should get a few decent developers together and write a For Dummies book titled: "How to spot a corporate consultant who's full of crap"

Great idea, but:

- Decent developers are hard to find.  The newbies in the industry have a good idea of which box to check or button to push, but they're clueless about what is actually going on.
- Decent developers (the old timers) don't even document.  They certainly won't write a book.  ;)
0
 
DanielcmorrisAuthor Commented:
awesome.  That's the first smile I've had on my face all day.  Thanks Kent.
0

Featured Post

Transaction-level recovery for Oracle database

Veeam Explore for Oracle delivers low RTOs and RPOs with agentless transaction log backup and transaction-level recovery of Oracle databases. You can restore the database to a precise point in time, even to a specific transaction.

  • 3
  • 3
Tackle projects and never again get stuck behind a technical roadblock.
Join Now