Set default LLM provider to OpenAI if no provider is specified; fix outer identifier check for Oracle dialect

Author: parthiv11

mindsdb/interfaces/agents/langchain_agent.py

@@ -89,6 +89,8 @@ def get_llm_provider(args: Dict) -> str:
         return "nvidia_nim"
     if args["model_name"] in GOOGLE_GEMINI_CHAT_MODELS:
         return "google"
+    else:
+        return 'openai'  # Default to OpenAI if no provider is specified
 
     # For vLLM, require explicit provider specification
     raise ValueError("Invalid model name. Please define a supported llm provider")

mindsdb/utilities/render/sqlalchemy_render.py

@@ -152,7 +152,7 @@ def to_column(self, identifier: ast.Identifier) -> sa.Column:
 
             parts2.append(part)
         text = '.'.join(parts2)
-        if identifier.is_outer and self.dialect.name == 'oracle':
+        if getattr(identifier, 'is_outer', False) and self.dialect.name == 'oracle':
             text += '(+)'
         return sa.column(text, is_literal=True)