Github user selvaganesang commented on a diff in the pull request:

    
https://github.com/apache/incubator-trafodion/pull/1232#discussion_r139270056
  
    --- Diff: core/sql/sqlcomp/nadefaults.cpp ---
    @@ -1336,6 +1336,8 @@ SDDkwd__(EXE_DIAGNOSTIC_EVENTS,               "OFF"),
     
      SDDui___(EXE_MEMORY_FOR_PROBE_CACHE_IN_MB,        "100"),
     
    + SDDui___(EXE_MEMORY_FOR_UNPACK_ROWS_IN_MB,        "100"),
    --- End diff --
    
    Because there is an easy way to change the memory size, we can determine 
the optimal default at a later date.  I believe the current default of 100 MB 
should be good enough for most of the use cases.


---

Reply via email to