Skip to content

Conversation

@lasley
Copy link
Member

@lasley lasley commented Sep 14, 2016

  • Add primary key to pat_id
  • Reorder Pks to match table definition order

SQL table definition:

USE [cph]
GO

/****** Object:  Table [dbo].[cp_acct]    Script Date: 9/14/2016 4:17:40 PM ******/
SET ANSI_NULLS ON
GO

SET QUOTED_IDENTIFIER ON
GO

CREATE TABLE [dbo].[cp_acct](
    [pat_id] [int] NOT NULL,
    [ID] [int] NOT NULL,
    [acct_type_cn] [int] NULL,
    [resp_pty_yn] [int] NULL,
    [chromis_id] [int] NULL,
 CONSTRAINT [PK_CP_ACCT] PRIMARY KEY CLUSTERED 
(
    [pat_id] ASC,
    [ID] ASC
)WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON, FILLFACTOR = 90) ON [PRIMARY]
) ON [PRIMARY]

GO

* Add primary key to pat_id
* Reorder Pks to match table definition order
@tedsalmon
Copy link
Contributor

Interesting; How are we going about tracking changes to the CarePoint database? Either way, 👍 on the code changes.

@tedsalmon tedsalmon merged commit 6fa70e7 into release/0.1 Sep 14, 2016
@tedsalmon tedsalmon deleted the bugfix/0.1/LABS-261_account-fix branch September 14, 2016 17:42
@lasley
Copy link
Member Author

lasley commented Sep 14, 2016

We aren't tracking (or performing) changes, I was providing their schema as proof for this change. Our connector is a bit finicky when it comes to identity lookups and creates. TBD how the update handles it, but I'll likely need to rework that method in the branch for this ticket in connector.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants