temp: show different approaches

edx_exams/apps/core/management/commands/bulk_add_course_staff.py

@@ -57,13 +57,17 @@ def handle(self, *args, **options):
 
         logger.info('Bulk add course staff complete!')
 
+    # todo: commented this out in the hopes it is not needed and would autocommit changes
     # @transaction.atomic
     def add_course_staff_from_csv(self, csv_file, batch_size, batch_delay):
         """
         Add the given set of course staff provided in csv
         """
         reader = list(csv.DictReader(csv_file))
 
+        #  todo: remove as needed. This is the new way I'm suggesting after research. I have added the batch size
+        #   (remove batch delay if going this route) and I've used get_or_create instead since the db call is required
+        #   anyway.
         User.objects.bulk_create(
             (User(
                 username=row.get('username'),
@@ -83,8 +87,9 @@ def add_course_staff_from_csv(self, csv_file, batch_size, batch_delay):
             batch_size=batch_size,
         )
 
-        users = {}
+        # users = {}
 
+        #  todo: remove as needed. This is the original way, but this requires a db call.
         # bulk create users
         # for i in range(0, len(reader), batch_size):
         #     User.objects.bulk_create(
@@ -95,6 +100,9 @@ def add_course_staff_from_csv(self, csv_file, batch_size, batch_delay):
         #         ignore_conflicts=True,
         #     )
 
+        # todo: remove as needed, this is the way I tried to get bulk create users return value and add to dictionary
+        #  to avoid the db call. Unfortunately this doesn't work as the return value doesn't include the PK for auto
+        #  fields
         # bulk_users = User.objects.bulk_create(
         #     (User(
         #         username=row.get('username'),