Fix issue-2293

[andyfengHKU]

Fix issue #2293. The problem is that we always assume internal ID to be non-NULL during update which is not true in case of OPTIONAL MATCH.

[codecov]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (9b569a4) 89.69% compared to head (89816a3) 89.69%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2298   +/-   ##
=======================================
  Coverage   89.69%   89.69%           
=======================================
  Files        1023     1023           
  Lines       35456    35474   +18     
=======================================
+ Hits        31801    31818   +17     
- Misses       3655     3656    +1     

Files	Coverage Δ
.../processor/operator/persistent/insert_executor.cpp	97.89% <100.00%> (+0.14%)	⬆️
src/processor/operator/persistent/set_executor.cpp	97.84% <100.00%> (+0.02%)	⬆️
src/storage/local_table.cpp	94.19% <100.00%> (+0.02%)	⬆️
src/storage/store/node_table.cpp	100.00% <100.00%> (ø)
src/storage/store/rel_table.cpp	95.84% <100.00%> (+0.05%)	⬆️
src/storage/index/hash_index.cpp	96.32% <50.00%> (-0.77%)	⬇️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.