src/main/java/org/opensearch/knn/index/codec/KNN80Codec/KNN80DocValuesConsumer.java (1 line):
	- line 134: //TODO: I think this can be refactored to avoid this copy and then write


src/main/java/org/opensearch/knn/index/util/KNNLibrary.java (1 line):
	- line 355: //TODO: To think about in future: for PQ, if dimension is not divisible by code count, PQ will fail. Right now,


src/main/java/org/opensearch/knn/index/memory/NativeMemoryCacheManager.java (1 line):
	- line 98: //TODO: Does this really need to be executed with an executor? Also, does invalidateAll really need to be


jni/src/jni_util.cpp (1 line):
	- line 444: //TODO: This potentially should use const char *


src/main/java/org/opensearch/knn/index/KNNSettings.java (1 line):
	- line 459: // TODO: replace cache-rebuild with index reload into the cache


jni/include/jni_util.h (1 line):
	- line 56: //TODO: My concern with this function is that it will make a lot of calls between the JVM. A few options


jni/src/faiss_wrapper.cpp (1 line):
	- line 236: //TODO make it different at search and write