Uploaded image for project: 'New Features 2017'
  1. New Features 2017
  2. NF-86

Review Performance Management Schema

    Details

    • Type: Support Activity
    • Status: Resolved
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Module:
      Performance Management
    • Reported by:
      Client
    • Issue Importance:
      Q1

      Description

      Please see attached schema creation script for the tables and constraints for the Performance management module.

        Attachments

          Issue Links

            Activity

            matthew.dominioni Matthew Dominioni (Inactive) created issue -
            Hide
            indraprakash Indraprakash Tiwari (Inactive) added a comment -

            I have reviewed attached script, and seem to be some changes required.
            1. We cannot use default constraint without specifying constraint name, constraint name must be specify, see tables script for (Surveys, Feedback, Metrics, PerformanceCheckins, etc)
            2. Cannot use TEXTIMAGE_ON when a table has no text, ntext, image, varchar(max), nvarchar(max), varbinary(max), xml or large CLR type columns. ( see tables script for (PerformanceRatings)
            3. [Created] [datetime] NOT NULL,
            [CreatedBy] [bigint] NOT NULL,
            [Modified] [datetime] NULL,
            [ModifiedBy] [bigint] NULL,
            These four columns should be present in all tables.
            4. IsCompanyUser column must be present in tables, which tell us user level (gloabal/client)
            5. Use varchar instead of nvarchar, wherever is possible
            6. Use Date datatype instead of datetime, where time is not required.

            Observation:
            Workterra using datetimestamp for created and UserID for createdby

            Show
            indraprakash Indraprakash Tiwari (Inactive) added a comment - I have reviewed attached script, and seem to be some changes required. 1. We cannot use default constraint without specifying constraint name, constraint name must be specify, see tables script for (Surveys, Feedback, Metrics, PerformanceCheckins, etc) 2. Cannot use TEXTIMAGE_ON when a table has no text, ntext, image, varchar(max), nvarchar(max), varbinary(max), xml or large CLR type columns. ( see tables script for (PerformanceRatings) 3. [Created] [datetime] NOT NULL, [CreatedBy] [bigint] NOT NULL, [Modified] [datetime] NULL, [ModifiedBy] [bigint] NULL, These four columns should be present in all tables. 4. IsCompanyUser column must be present in tables, which tell us user level (gloabal/client) 5. Use varchar instead of nvarchar, wherever is possible 6. Use Date datatype instead of datetime, where time is not required. Observation: Workterra using datetimestamp for created and UserID for createdby
            indraprakash Indraprakash Tiwari (Inactive) made changes -
            Field Original Value New Value
            Assignee Indraprakash Tiwari [ indraprakash ] Matthew Dominioni [ matthew.dominioni ]
            shyam.sharma shyam sharma (Inactive) made changes -
            Issue Type Task [ 3 ] Support Activity [ 10301 ]
            Status To Do [ 10000 ] Open [ 1 ]
            Workflow Software Simplified Workflow for Project NF [ 31470 ] WT_Support [ 31486 ]
            Hide
            indraprakash Indraprakash Tiwari (Inactive) added a comment - - edited

            We can not use "*" in any query and stored procedure, Colum name must be specify.
            see below procedure
            PM_SP_Common_IsEmployeeIdInManagementChain, IsEmployeePositionIdInManagementChain

            Show
            indraprakash Indraprakash Tiwari (Inactive) added a comment - - edited We can not use "*" in any query and stored procedure, Colum name must be specify. see below procedure PM_SP_Common_IsEmployeeIdInManagementChain, IsEmployeePositionIdInManagementChain
            Hide
            matthew.dominioni Matthew Dominioni (Inactive) added a comment -

            I understand all of the requested changes except for "4. IsCompanyUser column must be present in tables...". Can you provide more details and maybe give an example of another table that has this column present and what it's used for?

            Show
            matthew.dominioni Matthew Dominioni (Inactive) added a comment - I understand all of the requested changes except for "4. IsCompanyUser column must be present in tables...". Can you provide more details and maybe give an example of another table that has this column present and what it's used for?
            matthew.dominioni Matthew Dominioni (Inactive) made changes -
            Link This issue blocks NF-88 [ NF-88 ]
            Hide
            matthew.dominioni Matthew Dominioni (Inactive) added a comment -

            review must be complete before checking in initial database scripts.

            Show
            matthew.dominioni Matthew Dominioni (Inactive) added a comment - review must be complete before checking in initial database scripts.
            matthew.dominioni Matthew Dominioni (Inactive) made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            matthew.dominioni Matthew Dominioni (Inactive) made changes -
            Attachment CB_x.x_PM_NewTables_Initial.sql [ 38498 ]
            matthew.dominioni Matthew Dominioni (Inactive) made changes -
            Hide
            matthew.dominioni Matthew Dominioni (Inactive) added a comment -

            I have updated attached script file. Once approved, I will split this out and check in individual files per naming convention one per table and the stored procedures.

            Show
            matthew.dominioni Matthew Dominioni (Inactive) added a comment - I have updated attached script file. Once approved, I will split this out and check in individual files per naming convention one per table and the stored procedures.
            matthew.dominioni Matthew Dominioni (Inactive) made changes -
            Assignee Matthew Dominioni [ matthew.dominioni ] Indraprakash Tiwari [ indraprakash ]
            Hide
            indraprakash Indraprakash Tiwari (Inactive) added a comment -

            Hi Matt,
            You can see the structure of employee table. IsCompanyUser having a flag which indicate that entries been modified by Company user or Admin/Partner.

            IsCompanyUser = 1 means changes did by Company User Only
            IsCompanyUser = 0 means changes did by Admin/partner which is exists on Onlineenrollment table.

            One more point, I have seen that you have dropped all the default constraint from schema, Can you verify once again whether it is required or not?

            Rest scripts are fine to me.

            Regards,
            Indra

            Show
            indraprakash Indraprakash Tiwari (Inactive) added a comment - Hi Matt, You can see the structure of employee table. IsCompanyUser having a flag which indicate that entries been modified by Company user or Admin/Partner. IsCompanyUser = 1 means changes did by Company User Only IsCompanyUser = 0 means changes did by Admin/partner which is exists on Onlineenrollment table. One more point, I have seen that you have dropped all the default constraint from schema, Can you verify once again whether it is required or not? Rest scripts are fine to me. Regards, Indra
            indraprakash Indraprakash Tiwari (Inactive) made changes -
            Assignee Indraprakash Tiwari [ indraprakash ] Matthew Dominioni [ matthew.dominioni ]
            Hide
            matthew.dominioni Matthew Dominioni (Inactive) added a comment -

            Checked in all scripts after requested changes have been made.

            Show
            matthew.dominioni Matthew Dominioni (Inactive) added a comment - Checked in all scripts after requested changes have been made.
            matthew.dominioni Matthew Dominioni (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status In Progress [ 3 ] Resolved [ 5 ]
            Hide
            matthew.dominioni Matthew Dominioni (Inactive) added a comment -

            Add all scripts into SVN trunk revision 78799

            Show
            matthew.dominioni Matthew Dominioni (Inactive) added a comment - Add all scripts into SVN trunk revision 78799
            rakeshr Rakesh Roy (Inactive) made changes -
            Issue Importance Q1 [ 14900 ]
            Transition Time In Source Status Execution Times
            shyam sharma (Inactive) made transition -
            To Do Open
            2h 43m 1
            Matthew Dominioni (Inactive) made transition -
            Open In Progress
            12h 39m 1
            Matthew Dominioni (Inactive) made transition -
            In Progress Resolved
            29d 20h 59m 1

              People

              Assignee:
              matthew.dominioni Matthew Dominioni (Inactive)
              Reporter:
              matthew.dominioni Matthew Dominioni (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: