Fix test reliability with table order (#5087)

Summary:
X-link: https://github.com/facebookresearch/FBGEMM/pull/2097

- Fix test reliability with table order

Pull Request resolved: #5087

Reviewed By: spcyppt

Differential Revision: D86242426

Pulled By: q10

fbshipit-source-id: 4ec307ff8fd9151bddb6bf7354bfe06f67a1fa0b

Author: q10

.github/workflows/_fbgemm_gpu_cuda_test.yml

@@ -132,6 +132,9 @@ jobs:
       #   clang-16: error: unknown argument: '-fno-tree-loop-vectorize'
       run: . $PRELUDE; install_cxx_compiler $BUILD_ENV gcc
 
+    - name: Install Build Tools
+      run: . $PRELUDE; install_build_tools $BUILD_ENV
+
     - name: Install CUDA
       run: . $PRELUDE; install_cuda $BUILD_ENV ${{ matrix.cuda-version }}
 

fbgemm_gpu/fbgemm_gpu/split_table_batched_embeddings_ops_training.py

@@ -1560,7 +1560,7 @@ def get_table_name_for_logging(table_names: Optional[list[str]]) -> str:
             return "<Unknown>"
         # Do this because sometimes multiple shards of the same table could appear
         # in one TBE.
-        table_name_set = set(table_names)
+        table_name_set = sorted(set(table_names))
         if len(table_name_set) == 1:
             return next(iter(table_name_set))
         return f"<{len(table_name_set)} tables>: {table_name_set}"

fbgemm_gpu/setup.py

@@ -4,8 +4,6 @@
 # This source code is licensed under the BSD-style license found in the
 # LICENSE file in the root directory of this source tree.
 
-# @licenselint-loose-mode
-
 import argparse
 import logging
 import os
@@ -655,7 +653,7 @@ def main(argv: list[str]) -> None:
         ]
         + [
             f"Programming Language :: Python :: {x}"
-            for x in ["3", "3.9", "3.10", "3.11", "3.12", "3.13"]
+            for x in ["3", "3.10", "3.11", "3.12", "3.13"]
         ],
     )
 

fbgemm_gpu/test/tbe/utils/split_embeddings_test.py

@@ -178,17 +178,17 @@ def test_get_table_name_for_logging(self) -> None:
             SplitTableBatchedEmbeddingBagsCodegen.get_table_name_for_logging(
                 ["t1", "t2"]
             ),
-            "<2 tables>: {'t1', 't2'}",
+            "<2 tables>: ['t1', 't2']",
         )
         self.assertEqual(
             SplitTableBatchedEmbeddingBagsCodegen.get_table_name_for_logging(
                 ["t1", "t2", "t1"]
             ),
-            "<2 tables>: {'t1', 't2'}",
+            "<2 tables>: ['t1', 't2']",
         )
         self.assertEqual(
             SplitTableBatchedEmbeddingBagsCodegen.get_table_name_for_logging([]),
-            "<0 tables>: set()",
+            "<0 tables>: []",
         )
 
     @unittest.skipIf(*gpu_unavailable)